-
Notifications
You must be signed in to change notification settings - Fork 359
Add feature flag scaffolding to support experiments restricted to a specific Flutter channel #9440
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with some minor comments.
/// | ||
/// https://github.com/flutter/devtools/issues/7854 | ||
static bool inspectorV2 = true; | ||
static final inspectorV2 = BooleanFeatureFlag( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here and below. If we don't need these feature flags anymore, we may as well remove them.
enabled: true, | ||
); | ||
|
||
/// Stores a map of all the feature flags for debugging purposes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't a map anymore :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Work towards #9438
Previously all feature flags in DevTools could only be controlled by a boolean. This PR allows us to specify a Flutter channel to enable a feature for (e.g.
beta
will enable the experiment if a user is on thebeta
channel or adev
branch of Flutter). See #9439 for why this is using Flutter instead of Dart channels.FYI @kevmoo @mdebbar @natebiggs - This will allow us to enable dart2wasm compilation for everyone on Flutter beta now (which will go out on Oct 15th) without having to wait for the next beta after Flutter 3.38 (which will go out on November 12th).