Fix Regression: interpreter selection status bar not working when switching environments #1126
Fix Regression: interpreter selection status bar not working when switching environments #1126StellaHuang95 wants to merge 1 commit intomicrosoft:mainfrom
Conversation
|
thinking on this and the edge cases here- thanks for the PR |
Thanks. Looking back at the original bug, the user reported that It seems reasonable to me that |
|
ended up doing a larger overhaul since this needed area in general needed some work. Incorporated your PR improvements into here: #1155 |
fixes #1124
The changes made in #1120 introduced logic to avoid writing unnecessary settings when the selected environment manager is the implicit default (system manager). This was intended to prevent polluting
settings.json.However, this optimization broke manual interpreter selection. When a user clicks the status bar and explicitly selects a global/system interpreter in a workspace with an empty
settings.json, the selection is silently ignored because the code determines "this is the default, so don't write it."This PR added an optional
forceWriteflag toEditAllManagerSettings,EditEnvManagerSettings, andEditPackageManagerSettingsinterfaces and updated all user-initiated callers inenvManagers.tsandenvCommands.tsto passforceWrite: true.