Change how we fetch previously set env vars

microsoft · Aug 3, 2023 · f6a14bb · f6a14bb
1 parent 43e5f6d
commit f6a14bb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -46,7 +46,7 @@
         "theme": "dark"
     },
     "engines": {
-        "vscode": "^1.79.0-20230526"
+        "vscode": "^1.81"
     },
     "enableTelemetry": false,
     "keywords": [

diff --git a/src/client/interpreter/activation/terminalEnvVarCollectionService.ts b/src/client/interpreter/activation/terminalEnvVarCollectionService.ts
@@ -164,13 +164,13 @@ export class TerminalEnvVarCollectionService implements IExtensionActivationServ
             }
         }
 
-        for (const key in previousEnv) {
-            // If the previous env var is not in the current env, clear it from collection.
+        envVarCollection.forEach((key) => {
+            // If a previously set env var is not in the current env, clear it from collection.
             if (!(key in env)) {
                 traceVerbose(`Clearing environment variable ${key} from collection`);
                 envVarCollection.delete(key);
             }
-        }
+        });
 
         const displayPath = this.pathUtils.getDisplayName(settings.pythonPath, workspaceFolder?.uri.fsPath);
         const description = new MarkdownString(`${Interpreters.activateTerminalDescription} \`${displayPath}\``);