Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 6 commits into from
Jul 22, 2022

Conversation

karrtikr
Copy link

Closes #19089

@karrtikr karrtikr added the feature-request Request for new features or functionality label Jul 20, 2022
@karrtikr karrtikr marked this pull request as ready for review July 21, 2022 21:53
@@ -18,6 +18,11 @@ export class ConfigurationService implements IConfigurationService {
this.workspaceService = this.serviceContainer.get<IWorkspaceService>(IWorkspaceService);
}

// 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