-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add mentions and links to honggfuzz-rs #14
base: master
Are you sure you want to change the base?
Conversation
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 agree that this is not true right now, but I also want to add that IIRC cargo-fuzz was meant to get support for other fuzzers eventually :)
* [Setup](https://crates.io/crates/honggfuzz#dependencies) | ||
* [Tutorial/Guide](https://crates.io/crates/honggfuzz#how-to-use-this-crate) | ||
* [API](https://docs.rs/honggfuzz) | ||
* [Related docs](https://crates.io/crates/honggfuzz#relevant-documentation-about-honggfuzz) |
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.
Oh? Does mdbook support external links in the TOC?
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 have no idea, I didn't test anything, just edited in github directly.
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.
Just tried rendering this branch with mdbook. It appears mdbook adds a .html
to the end of each of the external links, causing them to break. We might just need to create a new dedicated page for hongfuzz within this book containing all these links
That's really fine by me to say that |
Also, I think we should remove the mention that
cargo-fuzz
is the recommended fuzzer in this file: https://github.com/rust-fuzz/book/blob/master/src/cargo-fuzz.md.I think there is two reasons for this:
closes #10