Skip to content

Commit 682d973

Browse files
authored
Include interperter name in message (#10174)
For #10071 Fix to ensure interpreter is included in message, here's the actual message seen by the user. Error: Jupyter cannot be started. Error attempting to locate jupyter: 'Kernelspec' module not installed in the selected interpreter ({0}). Note the {0}
1 parent 7d285ec commit 682d973

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/client/datascience/jupyter/interpreter/jupyterInterpreterSubCommandExecutionService.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ export class JupyterInterpreterSubCommandExecutionService
9292
}
9393

9494
if (productsNotInstalled.length === 1 && productsNotInstalled[0] === Product.kernelspec) {
95-
return DataScience.jupyterKernelSpecModuleNotFound();
95+
return DataScience.jupyterKernelSpecModuleNotFound().format(interpreter.path);
9696
}
9797

9898
return getMessageForLibrariesNotInstalled(productsNotInstalled, interpreter.displayName);

src/test/datascience/jupyter/interpreter/jupyterInterpreterSubCommandExecutionService.unit.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,7 @@ suite('Data Science - Jupyter InterpreterSubCommandExecutionService', () => {
281281
undefined
282282
);
283283

284-
assert.equal(reason, DataScience.jupyterKernelSpecModuleNotFound());
284+
assert.equal(reason, DataScience.jupyterKernelSpecModuleNotFound().format(selectedJupyterInterpreter.path));
285285
});
286286
test('Can start jupyer notebook', async () => {
287287
const output = await jupyterInterpreterExecutionService.startNotebook([], {});

0 commit comments

Comments
 (0)