Skip to content

Conversation

@sgrekhov
Copy link
Contributor

@sgrekhov sgrekhov commented Apr 4, 2025

No description provided.

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, couple of typos!

@sgrekhov
Copy link
Contributor Author

Missed this one, sorry. Updated. PTAL.

@sgrekhov sgrekhov requested a review from eernstg April 24, 2025 06:43
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 (added a few missing backquotes).

@eernstg eernstg merged commit 3423649 into dart-lang:master Apr 24, 2025
2 checks passed
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Apr 25, 2025
2025-04-25 sgrekhov22@gmail.com dart-lang/co19#3122. Expect an error in case of non-constant in a constant expression (dart-lang/co19#3159)
2025-04-24 sgrekhov22@gmail.com dart-lang/co19#3057. Add return statement in loops tests (dart-lang/co19#3158)
2025-04-24 sgrekhov22@gmail.com dart-lang/co19#3057. Update instance check tests and add negated instance check ones (dart-lang/co19#3131)
2025-04-23 sgrekhov22@gmail.com dart-lang/co19#3057. Add promotion tests for C-style for-loop (dart-lang/co19#3153)
2025-04-18 sgrekhov22@gmail.com Fixes dart-lang/co19#3122. Update constants evaluation according to the specification (dart-lang/co19#3150)

Cq-Include-Trybots: luci.dart.try:analyzer-linux-release-try
Change-Id: Ie89c9a3fbb920cb7e6488d652118ccc805b6f4ad
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/424760
Commit-Queue: Erik Ernst <eernst@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
Commit-Queue: Alexander Thomas <athom@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.

2 participants