microsoft / vscode-cpptools

Official repository for the Microsoft C/C++ extension for VS Code.
Other
5.42k stars 1.52k forks source link

Add a "default" value to C_Cpp.experimentalFeatures (i.e. make the current default of "disabled" actually disable it) #12163

Open AlexandraKemperMS opened 3 months ago

AlexandraKemperMS commented 3 months ago

Feature Request

At this point, we have overloaded the "Experimental Features" setting and need a separate setting for A/B experimentation. The experimental features setting was originally created to roll out beta features to users. image

With the A/B testing, we need a new setting. This setting should have three options:

Since we usually do not have multiple A/B experiments running at once, we do not need multiple settings to opt in/out (like Python has). We can wordsmith what the descriptions would say with the UI/UX team.

sean-mcmanus commented 3 months ago

I'm not understanding why we need a new setting and can't just change the existing setting's setting from "disabled" to "default" (a new 3rd value), i.e. I think we should do that instead. The meaning of the existing experimentalFeatures settings seems valid still and not really in conflict with the A/B experiment.

AlexandraKemperMS commented 3 months ago

When I talked to Bob about this he had mentioned that we still want to keep the original setting since we might have "beta" features that we want to release as an experimental feature. These are separate from A/B experiments. You can see other extensions do this as well: image

sean-mcmanus commented 3 months ago

@AlexandraKemperMS @bobbrow I'm still not understanding. We can still have beta non-A/B experimental features with the one setting still.

github-actions[bot] commented 1 month ago

This feature request is being closed due to insufficient upvotes. Please leave a 👍-upvote or 👎-downvote reaction on the issue to help us prioritize it. When enough upvotes are received, this issue will be eligible for our backlog.