From 7c91a03543ed9232ed387b59bad7ec5dc44c0c92 Mon Sep 17 00:00:00 2001 From: Jah-yee <166608075+Jah-yee@users.noreply.github.com> Date: Sat, 7 Nov 2026 10:21:16 +0800 Subject: [PATCH] fix: correct typos in comments and error messages Fixes four spelling mistakes in comments and one error string: - configSettings.ts: 'propogate' -> 'propagate' - condaInheritEnvPrompt.ts: 'intialize' -> 'initialize' - condaService.ts: 'wether' -> 'whether' - testBootstrap.ts: 'succesfully' -> 'successfully' --- src/client/common/configSettings.ts | 2 +- src/client/interpreter/virtualEnvs/condaInheritEnvPrompt.ts | 2 +- .../common/environmentManagers/condaService.ts | 2 +- src/client/repl/replCommandHandler.ts | 2 +- .../interpreterSelector/commands/setInterpreter.unit.test.ts | 2 +- src/test/index.ts | 2 +- src/test/performanceTest.ts | 2 +- src/test/testBootstrap.ts | 2 +- .../common/managers/testConfigurationManager.unit.test.ts | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/client/common/configSettings.ts b/src/client/common/configSettings.ts index 91c06d9331fd..52a9d7ab7c5b 100644 --- a/src/client/common/configSettings.ts +++ b/src/client/common/configSettings.ts @@ -390,7 +390,7 @@ export class PythonSettings implements IPythonSettings { this.experiments = experiments; } // Note we directly access experiment settings using workspace service in ExperimentService class. - // Any changes here specific to these settings should propogate their as well. + // Any changes here specific to these settings should propagate their as well. this.experiments = this.experiments ? this.experiments : { diff --git a/src/client/interpreter/virtualEnvs/condaInheritEnvPrompt.ts b/src/client/interpreter/virtualEnvs/condaInheritEnvPrompt.ts index 6b5295724449..39b18b44b300 100644 --- a/src/client/interpreter/virtualEnvs/condaInheritEnvPrompt.ts +++ b/src/client/interpreter/virtualEnvs/condaInheritEnvPrompt.ts @@ -33,7 +33,7 @@ export class CondaInheritEnvPrompt implements IExtensionActivationService { this.initializeInBackground(resource).ignoreErrors(); } - @traceDecoratorError('Failed to intialize conda inherit env prompt') + @traceDecoratorError('Failed to initialize conda inherit env prompt') public async initializeInBackground(resource: Uri): Promise { const show = await this.shouldShowPrompt(resource); if (!show) { diff --git a/src/client/pythonEnvironments/common/environmentManagers/condaService.ts b/src/client/pythonEnvironments/common/environmentManagers/condaService.ts index 0aa91bdbfb45..226d3c9807f9 100644 --- a/src/client/pythonEnvironments/common/environmentManagers/condaService.ts +++ b/src/client/pythonEnvironments/common/environmentManagers/condaService.ts @@ -54,7 +54,7 @@ export class CondaService implements ICondaService { } } - return { path: activatePath, type: 'local' }; // return the default activate script wether it exists or not. + return { path: activatePath, type: 'local' }; // return the default activate script whether it exists or not. } /** diff --git a/src/client/repl/replCommandHandler.ts b/src/client/repl/replCommandHandler.ts index 4c856734c2ca..0642ead39a02 100644 --- a/src/client/repl/replCommandHandler.ts +++ b/src/client/repl/replCommandHandler.ts @@ -37,7 +37,7 @@ export async function openInteractiveREPL( viewColumn = existingReplViewColumn ?? viewColumn; alreadyExists = true; } else if (!notebookDocument) { - // Case where NotebookDocument doesnt exist, or + // Case where NotebookDocument doesn't exist, or // became outdated (untitled.ipynb created without Python extension knowing, effectively taking over original Python REPL's URI) notebookDocument = await openNotebookDocument('jupyter-notebook'); } diff --git a/src/test/configuration/interpreterSelector/commands/setInterpreter.unit.test.ts b/src/test/configuration/interpreterSelector/commands/setInterpreter.unit.test.ts index 7837245ec9d2..5cb5441d1923 100644 --- a/src/test/configuration/interpreterSelector/commands/setInterpreter.unit.test.ts +++ b/src/test/configuration/interpreterSelector/commands/setInterpreter.unit.test.ts @@ -1126,7 +1126,7 @@ suite('Set Interpreter Command', () => { Telemetry._resetSharedProperties(); }); - test('A telemetry event should be sent after manual entry of an intepreter path', async () => { + test('A telemetry event should be sent after manual entry of an interpreter path', async () => { const state: InterpreterStateArgs = { path: undefined, workspace: undefined }; const multiStepInput = TypeMoq.Mock.ofType>(); multiStepInput diff --git a/src/test/index.ts b/src/test/index.ts index b14dd212e70c..199ac445aadb 100644 --- a/src/test/index.ts +++ b/src/test/index.ts @@ -43,7 +43,7 @@ process.on('unhandledRejection', (ex: any, _a) => { }); /** - * Configure the test environment and return the optoins required to run moch tests. + * Configure the test environment and return the options required to run moch tests. */ function configure(): SetupOptions { process.env.VSC_PYTHON_CI_TEST = '1'; diff --git a/src/test/performanceTest.ts b/src/test/performanceTest.ts index 2398f745c27a..1b1f511a1e1f 100644 --- a/src/test/performanceTest.ts +++ b/src/test/performanceTest.ts @@ -5,7 +5,7 @@ /* Comparing performance metrics is not easy (the metrics can and always get skewed). -One approach is to run the tests multile times and gather multiple sample data. +One approach is to run the tests multiple times and gather multiple sample data. For Extension activation times, we load both extensions x times, and re-load the window y times in each x load. I.e. capture averages by giving the extensions sufficient time to warm up. This block of code merely launches the tests by using either the dev or release version of the extension, diff --git a/src/test/testBootstrap.ts b/src/test/testBootstrap.ts index ab902255203b..48fe57d5f428 100644 --- a/src/test/testBootstrap.ts +++ b/src/test/testBootstrap.ts @@ -29,7 +29,7 @@ Final solution: * Send a message to the socket server with a flag (true/false whether tests passed/failed) * Socket server (main procecss) will receive the test status flag. * This will kill the spawned process - * This main process will kill itself with exit code 0 if tests pass succesfully, else 1. + * This main process will kill itself with exit code 0 if tests pass successfully, else 1. */ const testFile = process.argv[2]; diff --git a/src/test/testing/common/managers/testConfigurationManager.unit.test.ts b/src/test/testing/common/managers/testConfigurationManager.unit.test.ts index 1b049d4f3fbe..5b46d7058234 100644 --- a/src/test/testing/common/managers/testConfigurationManager.unit.test.ts +++ b/src/test/testing/common/managers/testConfigurationManager.unit.test.ts @@ -54,7 +54,7 @@ suite('Unit Test Configuration Manager (unit)', () => { ); }); - test('Enabling a test product shoud disable other products', async () => { + test('Enabling a test product should disable other products', async () => { UNIT_TEST_PRODUCTS.filter((item) => item !== product).forEach((productToDisable) => { configService .setup((c) => c.disable(TypeMoq.It.isValue(workspaceUri), TypeMoq.It.isValue(productToDisable)))