Skip to content

Commit 2ce743d

Browse files
authored
Remove unnecessary kCliAnimationsFeatureName that is available as .configSetting. (#158013)
... for consistency with the rest of the file/features.
1 parent 49ccfb7 commit 2ce743d

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

packages/flutter_tools/lib/src/features.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -146,14 +146,12 @@ const Feature flutterCustomDevicesFeature = Feature(
146146
),
147147
);
148148

149-
const String kCliAnimationsFeatureName = 'cli-animations';
150-
151149
/// The [Feature] for CLI animations.
152150
///
153151
/// The TERM environment variable set to "dumb" turns this off.
154152
const Feature cliAnimation = Feature.fullyEnabled(
155153
name: 'animations in the command line interface',
156-
configSetting: kCliAnimationsFeatureName,
154+
configSetting: 'cli-animations',
157155
);
158156

159157
/// Enable native assets compilation and bundling.

packages/flutter_tools/test/general.shard/reporting/first_run_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ void main() {
2020
testWithoutContext('FirstRunMessenger informs user how to disable animations', () {
2121
final FirstRunMessenger messenger = setUpFirstRunMessenger(redisplayWelcomeMessage: false);
2222

23-
expect(messenger.licenseTerms, contains('flutter config --no-$kCliAnimationsFeatureName'));
23+
expect(messenger.licenseTerms, contains('flutter config --no-${cliAnimation.configSetting}'));
2424
});
2525

2626
testWithoutContext('FirstRunMessenger requires redisplay if it has never been run before', () {

0 commit comments

Comments
 (0)