-
Notifications
You must be signed in to change notification settings - Fork 28
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
More documentation #4
Conversation
- Change the description from something rustc-specific to something more general, outlining its advantages and disadvantages (and linking to graphs) - Move the important parts to the crate doc instead of the struct's doc, which makes it easier to find - Add inline Rustdoc examples for using it with HashMap and HashSet - Put the test in a module (not important, but it's what people expect to see)
r? @SimonSapin |
r=me the version number bumped to 1.0.1 to publish on crates.io, where Servo can pick it up in order to have the docs show up on doc.servo.org. |
@bors-servo r=SimonSapin |
|
I fixed the repo collaborators to address point 2. |
|
||
```toml | ||
[dependencies] | ||
fnv = "1.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be bumped to 1.0.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any reason to bump this everytime we release a new 1.0.x
release
Rebase of #3 to work with stable and beta. I couldn't get
cfg_attr!()
to work correctly with conditionally turning on features, so I just disabled the nightly-only tests for now.Fixes #3.