+
Skip to content

Conversation

elliette
Copy link
Member

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 the beta channel or a dev 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).

@elliette elliette requested a review from srawlins September 18, 2025 23:27
@elliette elliette requested review from a team, bkonyi and kenzieschmoll as code owners September 18, 2025 23:27
Copy link
Contributor

@bkonyi bkonyi left a 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(
Copy link
Contributor

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.
Copy link
Contributor

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 :)

Copy link
Contributor

@srawlins srawlins left a comment

Choose a reason for hiding this comment

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

LGTM!

@elliette elliette merged commit 98a52a8 into flutter:master Sep 24, 2025
67 of 69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载