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: re-classify configureOnOpen setting #4150

Merged
merged 2 commits into from
Oct 31, 2024

Conversation

rzhao271
Copy link
Contributor

@rzhao271 rzhao271 commented Oct 29, 2024

VS Code 1.95.0 is now out, and with it comes changes to the tagging system.

This PR changes the tags of the cmake.configureOnOpen setting.

For reference, the preview tag indicates that the setting ID has been finalized and that the setting is almost stable, the onExP tag indicates that the setting can be referenced in an ExP experiment, and the experimental tag, which this PR removes, now denotes a WIP setting that could potentially be renamed or removed later. If a setting is stable and not controlled by an experiment, it does not need any of these tags.

Let me know if the tags I suggest in the diff are inaccurate.

@gcampbell-msft
Copy link
Collaborator

@rzhao271 We feel confident with the state of that setting, does that mean that we can simply remove the tags for that setting?

@rzhao271
Copy link
Contributor Author

If it's not in an A/B experiment and it's ready to go, then you can remove all tags from the setting 🏁

@gcampbell-msft gcampbell-msft enabled auto-merge (squash) October 31, 2024 18:02
@gcampbell-msft gcampbell-msft merged commit c3a5713 into microsoft:main Oct 31, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants