Skip to content

Conversation

@sgrekhov
Copy link
Contributor

No description provided.

Copy link
Member

@stereotype441 stereotype441 left a comment

Choose a reason for hiding this comment

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

lgtm, thank you!

Copy link
Member

@eernstg eernstg left a comment

Choose a reason for hiding this comment

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

Looks good! I think it would be helpful to deal with language versions more explicitly, though.

Copy link
Contributor Author

@sgrekhov sgrekhov left a comment

Choose a reason for hiding this comment

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

Thank you. Updated. PTAL

@sgrekhov sgrekhov requested a review from eernstg April 15, 2025 09:08
Copy link
Member

@eernstg eernstg left a comment

Choose a reason for hiding this comment

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

LGTM

@eernstg eernstg merged commit 5601323 into dart-lang:master Apr 15, 2025
2 checks passed
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Apr 22, 2025
2025-04-16 sgrekhov22@gmail.com dart-lang/co19#3057. Add more try-catch(-finally) tests (dart-lang/co19#3149)
2025-04-16 sgrekhov22@gmail.com dart-lang/co19#3057. Add more flow analysis tests for try-catch and try-finally (dart-lang/co19#3142)
2025-04-15 sgrekhov22@gmail.com Fixes dart-lang/co19#3143. Update assertions, add language version, add new tests (dart-lang/co19#3144)

Cq-Include-Trybots: luci.dart.try:analyzer-linux-release-try
Change-Id: Id1491e304f02d580a42d1c3d9831b96eaca52fc6
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/423280
Commit-Queue: Erik Ernst <eernst@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants