Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #2996. Update CFE expected errors locations #2999

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

sgrekhov
Copy link
Contributor

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.

LGTM

@eernstg eernstg merged commit 4c0cf74 into dart-lang:master Nov 27, 2024
2 checks passed
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Dec 1, 2024
2024-11-28 sgrekhov22@gmail.com dart-lang/co19#2976. Update assertion texts according to the new spec (dart-lang/co19#3000)
2024-11-28 sgrekhov22@gmail.com dart-lang/co19#2119. Add issue number to other_declarations_A04_t07.dart (dart-lang/co19#3001)
2024-11-28 sgrekhov22@gmail.com dart-lang/co19#2976. Add semantics tests (dart-lang/co19#2998)
2024-11-27 sgrekhov22@gmail.com Fixes dart-lang/co19#2996. Update CFE expected errors locations (dart-lang/co19#2999)
2024-11-25 sgrekhov22@gmail.com dart-lang/co19#2559. Add extension types and enums to augmenting constructors test. Part 2. (dart-lang/co19#2972)
2024-11-25 sgrekhov22@gmail.com dart-lang/co19#2956. Add more type `void` tests (dart-lang/co19#2997)

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

2 participants