Skip to content
This repository was archived by the owner on Nov 20, 2024. It is now read-only.

Report unrelated argument types on Enums #3695

Merged
merged 1 commit into from
Sep 19, 2022
Merged

Report unrelated argument types on Enums #3695

merged 1 commit into from
Sep 19, 2022

Conversation

srawlins
Copy link
Contributor

@srawlins srawlins commented Sep 16, 2022

Description

If Iterable.contains or List.remove was called from within an enum, with an implicit receiver, it was not caught by iterable_contains_unrelated_type or list_remove_unrelated_type.

Fixes dart-lang/sdk#58876

@coveralls
Copy link

Coverage Status

Coverage increased (+0.001%) to 95.741% when pulling ecbe34b on unrelated-enum into cd1fe53 on main.

@srawlins
Copy link
Contributor Author

This passes TGP. Still need to test Flutter.

@srawlins
Copy link
Contributor Author

This passed a flutter presubmit as well.

@pq
Copy link
Contributor

pq commented Sep 19, 2022

Excellent!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

proposal: list_remove_unrelated_type and iterable_contains_unrelated_type
3 participants