Skip to content

Lint comparison to empty slice using PartialEq methods#13432

Merged
bors merged 1 commit into
rust-lang:masterfrom
samueltardieu:issue-13401
Sep 23, 2024
Merged

Lint comparison to empty slice using PartialEq methods#13432
bors merged 1 commit into
rust-lang:masterfrom
samueltardieu:issue-13401

Conversation

@samueltardieu

Copy link
Copy Markdown
Member

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

Fix #13401

@rustbot

rustbot commented Sep 21, 2024

Copy link
Copy Markdown
Collaborator

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

@GuillaumeGomez GuillaumeGomez left a comment

Copy link
Copy Markdown
Member

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

bors commented Sep 22, 2024

Copy link
Copy Markdown
Contributor

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

@samueltardieu

samueltardieu commented Sep 23, 2024

Copy link
Copy Markdown
Member Author

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

@Manishearth

Copy link
Copy Markdown
Member

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

@bors r+

@bors

bors commented Sep 23, 2024

Copy link
Copy Markdown
Contributor

📌 Commit acff511 has been approved by Manishearth

It is now in the queue for this repository.

@bors

bors commented Sep 23, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit acff511 with merge 7901289...

@bors

bors commented Sep 23, 2024

Copy link
Copy Markdown
Contributor

☀️ 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
@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