Skip to content

Fixes #3210. Don't expect promotion from FutureOr<void> to Future<void> or void#3211

Merged
eernstg merged 1 commit intodart-lang:masterfrom
sgrekhov:co19-3210
Jun 4, 2025
Merged

Fixes #3210. Don't expect promotion from FutureOr<void> to Future<void> or void#3211
eernstg merged 1 commit intodart-lang:masterfrom
sgrekhov:co19-3210

Conversation

@sgrekhov
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
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, thanks!

Copy link
Copy Markdown
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.

I think the description in A03_t13 needs to be adjusted. I'd suggest that line 37 expects s to have type Never, and then we'll wait for input from @stereotype441.

Comment thread TypeSystem/flow-analysis/promotion_via_type_test_A03_t13.dart
@sgrekhov sgrekhov added the needs-info Additional information needed from the issue author label Jun 2, 2025
Copy link
Copy Markdown
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

Comment thread TypeSystem/flow-analysis/promotion_via_type_test_A03_t13.dart
@eernstg eernstg merged commit 058a238 into dart-lang:master Jun 4, 2025
2 checks passed
copybara-service Bot pushed a commit to dart-lang/sdk that referenced this pull request Jun 6, 2025
2025-06-06 sgrekhov22@gmail.com Fixes dart-lang/co19#3217. Ignore warning in string_length_t07.dart (dart-lang/co19#3218)
2025-06-06 sgrekhov22@gmail.com dart-lang/co19#3180. Add tests for JSAnyOperatorExtension. Part 1. (dart-lang/co19#3215)
2025-06-04 sgrekhov22@gmail.com Fixes dart-lang/co19#3210. Don't expect promotion from `FutureOr<void>` to `Future<void>` or `void` (dart-lang/co19#3211)
2025-06-03 sgrekhov22@gmail.com Fixes dart-lang/co19#3192. Add tests for  constant expression `String?.length` (dart-lang/co19#3209)

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

needs-info Additional information needed from the issue author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants