Skip to content

Added setting to control when interpreter information is displayed in the status bar#19513

Merged
karrtikr merged 6 commits intomicrosoft:mainfrom
karrtikr:interpreterinofsetting
Jul 22, 2022
Merged

Added setting to control when interpreter information is displayed in the status bar#19513
karrtikr merged 6 commits intomicrosoft:mainfrom
karrtikr:interpreterinofsetting

Conversation

@karrtikr
Copy link

Closes #19089

@karrtikr karrtikr added the feature-request Request for new features or functionality label Jul 20, 2022
@karrtikr karrtikr force-pushed the interpreterinofsetting branch from a968e23 to f97a242 Compare July 21, 2022 18:53
@karrtikr karrtikr marked this pull request as ready for review July 21, 2022 21:53
}

// eslint-disable-next-line class-methods-use-this
public get onDidChange(): Event<ConfigurationChangeEvent | undefined> {
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Config service also updates itself on workspace config change, but often workspace config change event (obtained via WorkspaceService) fires before config service updates. So add a separate reliable change event for config service.

@karrtikr karrtikr added the skip package*.json package.json and package-lock.json don't both need updating label Jul 21, 2022
@karrtikr karrtikr merged commit 9b6bb7e into microsoft:main Jul 22, 2022
@karrtikr karrtikr deleted the interpreterinofsetting branch July 22, 2022 04:15
wesm pushed a commit to posit-dev/positron that referenced this pull request Mar 28, 2024
… the status bar (microsoft/vscode-python#19513)

* Added setting to control when interpreter information is displayed in the status bar

* Oops

* Team reviews

* Add feature

* Debounce change notification

* Add test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature-request Request for new features or functionality skip package*.json package.json and package-lock.json don't both need updating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide a setting to always show the selected interpreter

2 participants