Skip to content

Conversation

Alexendoo
Copy link
Member

Adds a lint to suggest using BuildHasher::hash_one

changelog: [manual_hash_one]: new lint

@rustbot
Copy link
Collaborator

rustbot commented Sep 23, 2023

r? @Manishearth

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 23, 2023
@Alexendoo Alexendoo force-pushed the manual-hash-one branch 2 times, most recently from fe8df96 to 7802b08 Compare September 23, 2023 18:33
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 25, 2023

📌 Commit 7802b08 has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 25, 2023

⌛ Testing commit 7802b08 with merge 3422282...

bors added a commit that referenced this pull request Sep 25, 2023
Add `manual_hash_one` lint

Adds a lint to suggest using [`BuildHasher::hash_one`](https://doc.rust-lang.org/std/hash/trait.BuildHasher.html#method.hash_one)

changelog: [`manual_hash_one`]: new lint
@bors
Copy link
Contributor

bors commented Sep 25, 2023

💔 Test failed - checks-action_test

@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 26, 2023

📌 Commit ec2f626 has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 26, 2023

⌛ Testing commit ec2f626 with merge 493ab53...

@bors
Copy link
Contributor

bors commented Sep 26, 2023

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

@bors bors merged commit 493ab53 into rust-lang:master Sep 26, 2023
@Alexendoo Alexendoo deleted the manual-hash-one branch September 26, 2023 19:43
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.

4 participants