Skip to content

Improve documentation on implicit_hasher lint #4322

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 2 commits into from
Aug 1, 2019
Merged

Conversation

j-maas
Copy link

@j-maas j-maas commented Aug 1, 2019

Provide an example of how to fix the implicit_hasher lint.

Fixes #3475.

changelog: none

Provide an example of how to fix the lint.
Copy link
Member

@phansch phansch left a comment

Choose a reason for hiding this comment

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

LGTM. r=me with the missing BuildHasher import fixed

@phansch
Copy link
Member

phansch commented Aug 1, 2019

@bors r+ thanks!

@bors
Copy link
Contributor

bors commented Aug 1, 2019

📌 Commit 5decadd has been approved by phansch

bors added a commit that referenced this pull request Aug 1, 2019
Improve documentation on implicit_hasher lint

Provide an example of how to fix the implicit_hasher lint.

Fixes #3475.

changelog: none
@bors
Copy link
Contributor

bors commented Aug 1, 2019

⌛ Testing commit 5decadd with merge d43ef7a...

@bors
Copy link
Contributor

bors commented Aug 1, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: phansch
Pushing d43ef7a to master...

@bors bors merged commit 5decadd into rust-lang:master Aug 1, 2019
@j-maas j-maas deleted the patch-1 branch August 2, 2019 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation for implicit_hasher is not very helpful
3 participants