Skip to content

[flutter_tools] add feature for iOS, android, and fuchsia - #61481

Merged
jonahwilliams merged 7 commits into
flutter:masterfrom
jonahwilliams:android_ios_fuchsia_features
Jul 16, 2020
Merged

jonahwilliams merged 7 commits into
flutter:masterfrom
jonahwilliams:android_ios_fuchsia_features

Conversation

@jonahwilliams

@jonahwilliams jonahwilliams commented Jul 14, 2020 •

Copy link
Copy Markdown
Contributor

Description

Add feature flags for android, ios, and fuchsia (on by default). After updating the g3 rollers, the fuchsia feature will be turned off by default. Creates a simpler base type of feature flags for g3 to extend.

Updates android, ios, fuchsia workflows to use feature flags check.
Removes concept of stable artifacts and checks on flutter version.

Fixes #58999
#52859
#12768

@fluttergithubbot fluttergithubbot added tool Affects the "flutter" command-line tool. See also t: labels. work in progress; do not review labels Jul 14, 2020
@jonahwilliams
jonahwilliams marked this pull request as ready for review July 15, 2020 20:01
@jonahwilliams
jonahwilliams requested a review from jmagman July 15, 2020 20:44

@jmagman jmagman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

After updating the g3 rollers, the fuchsia feature will be turned off by default.

I'm guessing this PR has to roll, then you'll immediately update to make fuchsia off? Definitely don't want our users to start by default caching fuchsia artifacts.


/// Whether fast single widget reloads are enabled.
bool get isSingleWidgetReloadEnabled => isEnabled(singleWidgetReload);
bool get isSingleWidgetReloadEnabled => false;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Want to put this one at the end of the list after the platforms? I thought for a second you edited this accidentally.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done!

import '../base/platform.dart';
import '../cache.dart';
import '../features.dart';
import '../globals.dart' as globals;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Woo!

import '../base/context.dart';
import '../base/platform.dart';
import '../doctor.dart';
import '../globals.dart' as globals;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Woo!


import '../base/platform.dart';
import '../doctor.dart';
import '../globals.dart' as globals;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Woo again!

@jonahwilliams

Copy link
Copy Markdown
Contributor Author

I'm guessing this PR has to roll, then you'll immediately update to make fuchsia off? Definitely don't want our users to start by default caching fuchsia artifacts.

Not quite - we still won't download fuchsia artifacts by default if precache is called, only android and iOS have that behavior. The feature is also disabled on dev, beta, stable channels so it isn't exposed any more than it was before

@jmagman

jmagman commented Jul 16, 2020

Copy link
Copy Markdown
Member
google3:///mobile/flutter/tools/lib/context.dart:798:9: Error: A constant constructor can't call a non-constant super constructor.
  const _Google3Features();

@jonahwilliams

Copy link
Copy Markdown
Contributor Author

noooooo

@jonahwilliams
jonahwilliams merged commit 859fce9 into flutter:master Jul 16, 2020
@jonahwilliams
jonahwilliams deleted the android_ios_fuchsia_features branch July 16, 2020 16:52
Pragya007 pushed a commit to Pragya007/flutter that referenced this pull request Aug 11, 2020
)

Add feature flags for android, ios, and fuchsia (on by default). After updating the g3 rollers, the fuchsia feature will be turned off by default. Creates a simpler base type of feature flags for g3 to extend.

Updates android, ios, fuchsia workflows to use feature flags check.
Removes concept of stable artifacts and checks on flutter version.

Fixes flutter#58999
flutter#52859
flutter#12768
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
)

Add feature flags for android, ios, and fuchsia (on by default). After updating the g3 rollers, the fuchsia feature will be turned off by default. Creates a simpler base type of feature flags for g3 to extend.

Updates android, ios, fuchsia workflows to use feature flags check.
Removes concept of stable artifacts and checks on flutter version.

Fixes flutter#58999
flutter#52859
flutter#12768
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding 'flutter config' for mobile

4 participants