-
Notifications
You must be signed in to change notification settings - Fork 326
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
Fix cargo doc #1788
Fix cargo doc #1788
Conversation
/// [`tendermint_light_client_verifier::options::Options`] struct for use in | ||
/// Tendermint-specific light client verification. |
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.
It would be better to shorten the text in the link:
/// [`Options`] struct for use in Tendermint-specific light client verification.
/// [`Options`]: tendermint_light_client_verifier::options::Options
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.
Hmm I tried and it don't seem like this style of link shortening works in Rustdoc.
//! [`ChainHandle`](crate::chain::handle::ChainHandle) can accept. | ||
//! These types trickle down to the | ||
//! [`ChainEndpoint`](crate::chain::ChainEndpoint) trait. |
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.
Good, but stylistically footnote-style markup reads better in code, like in my suggestion above.
* Fix cargo doc errors * Fix cargo doc * Add CI for cargo doc * Fix cargo doc * Fix Markdown link * Fix review feedback
Closes: #1776
Description
Todo:
PR author checklist:
unclog
.docs/
).Reviewer checklist:
Files changed
in the GitHub PR explorer.