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

Fix bad suggestion on option_if_let_else when there is complex subpat #8086

Merged
merged 1 commit into from
Dec 7, 2021

Conversation

dswij
Copy link
Member

@dswij dswij commented Dec 7, 2021

closes #7991

Prefer not warning any complex subpat in option_if_let_else rather than suggesting obscure suggestions.

changelog: [option_if_let_else] does not warn when complex subpat is present

@rust-highfive
Copy link

r? @giraffate

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 7, 2021
@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Dec 7, 2021

📌 Commit b5fa5b3 has been approved by giraffate

@bors
Copy link
Contributor

bors commented Dec 7, 2021

⌛ Testing commit b5fa5b3 with merge 86cea73...

@bors
Copy link
Contributor

bors commented Dec 7, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing 86cea73 to master...

@bors bors merged commit 86cea73 into rust-lang:master Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

option_if_let_else incorrect suggestion, ignores semantics of existing code
4 participants