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

fix: tag non-stable settings with new system #24287

Merged
merged 2 commits into from
Oct 11, 2024
Merged

Conversation

rzhao271
Copy link

@rzhao271 rzhao271 commented Oct 10, 2024

This PR changes some setting tags as follows:

  • Remove the experimental tag from the python.missingPackage.severity setting considering the setting does not seem to be experimental and that the setting does not seem to be part of any experiment.
  • Add the onExP tag onto the python.locator setting considering the setting is both experimental (based on its description) and part of an experiment.
  • Change the experimental tag to preview for python.terminal.shellIntegration.enabled, considering it is a non-stable setting that does not seem to be part of an experiment.
  • Add the onExP tag to python.REPL.sendToNativeREPL considering it is part of an experiment.

@rzhao271 rzhao271 added this to the October 2024 milestone Oct 10, 2024
@rzhao271 rzhao271 self-assigned this Oct 10, 2024
Copy link
Author

@rzhao271 rzhao271 left a comment

Choose a reason for hiding this comment

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

Please comment and set the PR to Changes Requested if you disagree with any of the new classifications.

@rzhao271 rzhao271 added debt Covers everything internal: CI, testing, refactoring of the codebase, etc. skip package*.json package.json and package-lock.json don't both need updating labels Oct 10, 2024
@@ -677,7 +675,8 @@
"scope": "resource",
"type": "boolean",
"tags": [
"experimental"
"experimental",
"onExP"
Copy link
Author

Choose a reason for hiding this comment

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

I just realized that VS Code stable wouldn't recognize the onExP tag, so I'm keeping the experimental tag for now instead of changing it to the new preview tag.

@karthiknadig karthiknadig requested review from luabud and cwebster-99 and removed request for anthonykim1 October 10, 2024 17:48
Copy link
Member

@luabud luabud left a comment

Choose a reason for hiding this comment

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

thank you so much!
after the next VS Code release we can probably remove the experimental tags, right?

@rzhao271 rzhao271 marked this pull request as ready for review October 10, 2024 19:09
@rzhao271 rzhao271 merged commit 9be9308 into main Oct 11, 2024
73 checks passed
@rzhao271 rzhao271 deleted the rzhao271/tag-settings branch October 11, 2024 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debt Covers everything internal: CI, testing, refactoring of the codebase, etc. 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.

2 participants