Skip to content

Negate suggestions when needed in bool_assert_comparison #10293

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

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

Alexendoo
Copy link
Member

changelog: none assuming this gets into the same release as #10218

Fixes #10291

r? @dswij

Thanks to @black-puppydog for spotting it early

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 4, 2023
Copy link
Member

@dswij dswij left a comment

Choose a reason for hiding this comment

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

Good find! Lgtm, thanks for this

@dswij
Copy link
Member

dswij commented Feb 8, 2023

@bors r+

@bors
Copy link
Contributor

bors commented Feb 8, 2023

📌 Commit 5546c82 has been approved by dswij

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Feb 8, 2023

⌛ Testing commit 5546c82 with merge fd2d8be...

@bors
Copy link
Contributor

bors commented Feb 8, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: dswij
Pushing fd2d8be to master...

1 similar comment
@bors
Copy link
Contributor

bors commented Feb 8, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: dswij
Pushing fd2d8be to master...

@bors bors merged commit fd2d8be into rust-lang:master Feb 8, 2023
@Alexendoo Alexendoo deleted the bool-assert-comparison-negation branch February 9, 2023 00:36
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.

Incorrect auto-fix: comparison to false breaks
4 participants