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

#2420. Exhaustiveness variable tests. Add switch statement check #2453

Merged
merged 2 commits into from
Jan 25, 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.

Blocked on dart-lang/language#3534.

Perhaps split this into one PR that uses return type String on a bunch of existing test functions containing switch expressions, and put the new tests into a different PR? We could then land the return-type part, and confine the blockedness to the second PR.

@eernstg eernstg added the status-blocked Blocked from making progress by another (referenced) issue label Dec 29, 2023
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.

I believe now this PR is nublocked. Please take another look

@sgrekhov sgrekhov requested a review from eernstg January 24, 2024 08:57
@sgrekhov sgrekhov removed the status-blocked Blocked from making progress by another (referenced) issue label Jan 24, 2024
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 db5ff89 into dart-lang:master Jan 25, 2024
2 checks passed
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Jan 26, 2024
2024-01-26 sgrekhov22@gmail.com dart-lang/co19#2485. Add constant evaluation tests for `Set` (dart-lang/co19#2509)
2024-01-26 sgrekhov22@gmail.com Fixes dart-lang/co19#2507. Fix roll failures (dart-lang/co19#2508)
2024-01-26 sgrekhov22@gmail.com dart-lang/co19#2485. Add is T constant evaluation tests (dart-lang/co19#2501)
2024-01-26 sgrekhov22@gmail.com dart-lang/co19#2485. Add `is! T` constant evaluation tests (dart-lang/co19#2504)
2024-01-25 sgrekhov22@gmail.com Fixes dart-lang/co19#2503. Update exhaustiveness_map_A02_t02.dart according to the flow analysis change (dart-lang/co19#2503)
2024-01-25 sgrekhov22@gmail.com dart-lang/co19#2420. Exhaustiveness variable tests. Add switch statement check (dart-lang/co19#2453)
2024-01-22 sgrekhov22@gmail.com dart-lang/co19#2485. Add more constant evaluation tests for record types (dart-lang/co19#2499)
2024-01-22 sgrekhov22@gmail.com dart-lang/co19#2420. Add more exhaustiveness tests (dart-lang/co19#2498)

Change-Id: If4af6cc1df4d44ad01e3578b8524748f67e7bac4
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/348660
Commit-Queue: Erik Ernst <eernst@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
Commit-Queue: Alexander Thomas <athom@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
sgrekhov added a commit to sgrekhov/co19 that referenced this pull request Feb 15, 2024
…heck (dart-lang#2453)

Exhaustiveness variable tests. Add switch statement check
sgrekhov added a commit to sgrekhov/co19 that referenced this pull request Feb 15, 2024
…heck (dart-lang#2453)

Exhaustiveness variable tests. Add switch statement check
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