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 comparison_chain false positive #5596

Merged
merged 1 commit into from
May 16, 2020
Merged

Conversation

ebroto
Copy link
Member

@ebroto ebroto commented May 13, 2020

changelog: comparison_chain: fix false positives when the binary operation is the same.

Fixes #5212

@phansch phansch self-requested a review May 14, 2020 05:48
@phansch phansch added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label May 14, 2020
@phansch
Copy link
Member

phansch commented May 16, 2020

@bors r+ thanks!

@bors
Copy link
Contributor

bors commented May 16, 2020

📌 Commit 9217675 has been approved by phansch

@bors
Copy link
Contributor

bors commented May 16, 2020

⌛ Testing commit 9217675 with merge 0c94273...

@bors
Copy link
Contributor

bors commented May 16, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: phansch
Pushing 0c94273 to master...

@bors bors merged commit 0c94273 into rust-lang:master May 16, 2020
@ebroto ebroto deleted the issue_5212 branch May 16, 2020 09:05
@csnover
Copy link

csnover commented May 20, 2020

Thank you for your work on fixing this bug!

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.

comparison_chain false positive when every test is the same comparison to a number
4 participants