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

Improve collapsible_match #6619

Merged
merged 3 commits into from
Jan 22, 2021
Merged

Conversation

camsteffen
Copy link
Contributor

changelog: Fix collapsible_match false negatives

Allow & and/or * on the binding and make sure the type still matches.

@rust-highfive
Copy link

r? @phansch

(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 Jan 22, 2021
@camsteffen
Copy link
Contributor Author

I tweaked my approach.

Copy link
Member

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

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

r=me, maybe the function could be clearer and/or moved into utils

clippy_lints/src/collapsible_match.rs Outdated Show resolved Hide resolved
@Manishearth
Copy link
Member

@bors delegate+

@bors
Copy link
Contributor

bors commented Jan 22, 2021

✌️ @camsteffen can now approve this pull request

@bors
Copy link
Contributor

bors commented Jan 22, 2021

⌛ Testing commit 3e0a088 with merge a949eeb...

bors added a commit that referenced this pull request Jan 22, 2021
Improve collapsible_match

changelog: Fix collapsible_match false negatives

Allow `&` and/or `*` on the binding and make sure the type still matches.
@flip1995
Copy link
Member

@bors r-

whoops, since when reacts bors to commands in discussion comments in code blocks? 😮

@bors
Copy link
Contributor

bors commented Jan 22, 2021

☀️ Try build successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Build commit: a949eeb (a949eeb5c93d8ab52c200ffab9efc5fb13214c67)

@camsteffen
Copy link
Contributor Author

@bors r+ and have a nice day

@bors
Copy link
Contributor

bors commented Jan 22, 2021

📌 Commit 50abde2 has been approved by camsteffen

@bors
Copy link
Contributor

bors commented Jan 22, 2021

⌛ Testing commit 50abde2 with merge 41d750c...

@bors
Copy link
Contributor

bors commented Jan 22, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: camsteffen
Pushing 41d750c to master...

@bors bors merged commit 41d750c into rust-lang:master Jan 22, 2021
@camsteffen
Copy link
Contributor Author

thanks @bors, ttyl

@camsteffen camsteffen deleted the collapsible-match branch July 8, 2021 21:47
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.

6 participants