Skip to content

Conversation

@samueltardieu
Copy link
Member

changelog: [comparison_to_empty]: Also detect comparaisons using PartialEq methods

Fix #13401

@rustbot
Copy link
Collaborator

rustbot commented Sep 21, 2024

r? @Manishearth

rustbot has assigned @Manishearth.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 21, 2024
Copy link
Member

@GuillaumeGomez GuillaumeGomez left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks! (still needs someone from the team to r+ it)

@bors
Copy link
Contributor

bors commented Sep 22, 2024

☔ The latest upstream changes (presumably #13440) made this pull request unmergeable. Please resolve the merge conflicts.

@samueltardieu
Copy link
Member Author

samueltardieu commented Sep 23, 2024

@Manishearth Did you forget to r+ it by chance?

@Manishearth
Copy link
Member

I think there was something else going on but it's not relevant anymore

@bors r+

@bors
Copy link
Contributor

bors commented Sep 23, 2024

📌 Commit acff511 has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 23, 2024

⌛ Testing commit acff511 with merge 7901289...

@bors
Copy link
Contributor

bors commented Sep 23, 2024

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

@bors bors merged commit 7901289 into rust-lang:master Sep 23, 2024
8 checks passed
@samueltardieu samueltardieu deleted the issue-13401 branch November 3, 2024 16:17
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_to_empty lint isn't triggered in case PartialEq::eq used to compare

5 participants