Skip to content

Conversation

@matthiaskrgr
Copy link
Member

look inside refs and detect if let &None = ...

Fixes #5396

changelog: redundant_pattern_matching: look inside Refs to fix FNs with "if let &None = .. "

look inside refs and detect if let &None = ...

Fixes rust-lang#5396

changelog:  redundant_pattern_matching: look inside Refs to fix FNs with "if let &None = .. "
@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 Mar 27, 2021
Copy link
Contributor

@giraffate giraffate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors r+

Thanks!

@matthiaskrgr
Copy link
Member Author

@giraffate unfortunately bors does not parse commands inside github pr review messages or something like that.
A normal pr message should work though.

@giraffate
Copy link
Contributor

@bors r+

(because bors doesn't start)

@bors
Copy link
Contributor

bors commented Mar 28, 2021

📌 Commit e006c77 has been approved by giraffate

@bors
Copy link
Contributor

bors commented Mar 28, 2021

⌛ Testing commit e006c77 with merge 4bcc552...

@giraffate
Copy link
Contributor

Oh, I didn't know that. Thanks @matthiaskrgr !

@bors
Copy link
Contributor

bors commented Mar 28, 2021

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

@bors bors merged commit 4bcc552 into rust-lang:master Mar 28, 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.

FN redundant_pattern_matching: reference to LHS op: &None

4 participants