Skip to content

Conversation

@cbracken
Copy link
Member

Flutter app development has been Dart 2 only for a significant amount of
time, and we're about to remove all support for Dart 1.

@sjindel-google
Copy link
Contributor

Don't we also need to update dev/devicelab/lib/tasks/microbenchmarks.dart (lines 78 - 82)?

@cbracken cbracken force-pushed the remove-dart1-benchmarks branch from c7b83f9 to e4f6362 Compare July 16, 2018 23:26
@cbracken
Copy link
Member Author

Don't we also need to update dev/devicelab/lib/tasks/microbenchmarks.dart (lines 78 - 82)?

Oops! Pushed before committing the non-manifest stuff. Updated.

Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@goderbauer
Copy link
Member

Looks like the analyzer has one tiny complaint.

Flutter app development has been Dart 2 only for a significant amount of
time, and we're about to remove all support for Dart 1.
@cbracken
Copy link
Member Author

Fixed.

@cbracken
Copy link
Member Author

@sjindel-google FYI the Flutter trees are currently closed pending the resolution of TODAY issues. I'll land this once the tree is re-opened.

@cbracken cbracken merged commit c0294e3 into flutter:master Jul 18, 2018
@cbracken cbracken deleted the remove-dart1-benchmarks branch July 18, 2018 22:56
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request Jul 19, 2018
Flutter app development has been Dart 2 only for a significant amount of
time, and we're about to remove all support for Dart 1.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants