Skip to content

Fix nonminimal_bool #[allow] attributes. #10588

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
Apr 2, 2023

Conversation

blyxyas
Copy link
Member

@blyxyas blyxyas commented Apr 2, 2023

Closes #10435
changelog: [nonminimal_bool]: Fix false-positive where the lint ignore #[allow] attributes.

r? @llogiq

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

blyxyas commented Apr 2, 2023

Sorry for taking so long to read the message.

@llogiq
Copy link
Contributor

llogiq commented Apr 2, 2023

Thank you! The implementation looks good and the test is appreciated. 👍

@bors r+

@bors
Copy link
Contributor

bors commented Apr 2, 2023

📌 Commit 36047b0 has been approved by llogiq

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Apr 2, 2023

⌛ Testing commit 36047b0 with merge 7fe83ed...

@bors
Copy link
Contributor

bors commented Apr 2, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing 7fe83ed to master...

@bors bors merged commit 7fe83ed into rust-lang:master Apr 2, 2023
@blyxyas blyxyas deleted the fix-allow_nonminimal_bool branch October 5, 2023 09:05
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.

Strange behavior re #[allow(clippy::nonminimal_bool)]
4 participants