Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

=> Dart 3.0! #4267

Merged
merged 3 commits into from
Apr 12, 2023
Merged

=> Dart 3.0! #4267

merged 3 commits into from
Apr 12, 2023

Conversation

pq
Copy link
Member

@pq pq commented Apr 8, 2023

With (internal) cl/523481067 landed and server on 3.0, I think we're actually good to go here and this is safe to land.

/cc @bwilkerson @srawlins

@pq pq marked this pull request as draft April 8, 2023 18:53
@github-actions github-actions bot added the linter-set-recommended Affects a rule in the recommended Dart rule set label Apr 8, 2023
@coveralls
Copy link

coveralls commented Apr 8, 2023

Coverage Status

Coverage: 96.43%. Remained the same when pulling 8ddc9e4 on dart3 into eb0ca50 on main.

@@ -193,4 +174,12 @@ class _Visitor extends SimpleAstVisitor<void> {

return true;
}

static TokenType _invertedTokenType(TokenType type) => switch (type) {
Copy link
Member

Choose a reason for hiding this comment

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

This looks so nice!

Copy link
Member Author

Choose a reason for hiding this comment

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

Bonus points for being generated by our assist. 🥳

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

For points, you could make this an extension on TokenType 😄

Copy link
Member Author

Choose a reason for hiding this comment

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

@pq pq changed the title [WIP] dart 3.0! => Dart 3.0! Apr 11, 2023
@pq pq marked this pull request as ready for review April 11, 2023 22:49
@pq pq merged commit 686e85d into main Apr 12, 2023
@pq pq deleted the dart3 branch April 12, 2023 18:00
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Aug 23, 2023
* dart 3.0 migration

* fixed dangling library
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
linter-set-recommended Affects a rule in the recommended Dart rule set
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants