-
Notifications
You must be signed in to change notification settings - Fork 144
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
docs(iroh-net): Update NodeAddr docs #2365
Conversation
This follows https://github.com/rust-lang/rfcs/blob/master/text/1574-more-api-documentation-conventions.md#appendix-a-full-conventions-text and updates the doc comments in an attempt to be more user-friendly.
/// [discovery]: https://docs.rs/iroh_net/*/iroh_net/index.html#node-discovery | ||
/// [home relay]: https://docs.rs/iroh_net/*/iroh_net/relay/index.html | ||
/// [Relay server]: https://docs.rs/iroh_net/*/iroh_net/index.html#relay-servers |
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.
As far as I'm aware this type of links are not verified when checking docs so, while these make sense, it would be good to avoid them in general
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.
@divagant-martian do you now how to link these without having to write them down like this? I first tried to write this as iroh_net#relay-servers
or even iroh_net
but I didn't know how to do inter-crate links and this was the only thing I could come up with.
}, | ||
} | ||
} | ||
|
||
/// Apply the options to `self`. |
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.
bit late but this one was missed
/// Apply the options to `self`. | ||
/// | ||
/// This is use to more tightly control the information stored in a [`NodeAddr`] |
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 is use to more tightly control the information stored in a [`NodeAddr`] | |
/// This is used to more tightly control the information stored in a [`NodeAddr`] |
/// Apply the options to `self`. | ||
/// Applies the options to `self`. | ||
/// | ||
/// This is use to more tightly control the information stored in ab [`AddrInfo`] |
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 is use to more tightly control the information stored in ab [`AddrInfo`] | |
/// This is used to more tightly control the information stored in ab [`AddrInfo`] |
## Description Some minor typos missed earlier. ## Breaking Changes <!-- Optional, if there are any breaking changes document them, including how to migrate older code. --> ## Notes & open questions Followup from #2365 ## Change checklist - [x] Self-review. - [x] Documentation updates if relevant. - ~~[ ] Tests if relevant.~~ - ~~[ ] All breaking changes documented.~~
## Description This follows https://github.com/rust-lang/rfcs/blob/master/text/1574-more-api-documentation-conventions.md#appendix-a-full-conventions-text and updates the doc comments in an attempt to be more user-friendly. ## Breaking Changes <!-- Optional, if there are any breaking changes document them, including how to migrate older code. --> ## Notes & open questions <!-- Any notes, remarks or open questions you have to make about the PR. --> ## Change checklist - [x] Self-review. - [x] Documentation updates if relevant. - ~~[ ] Tests if relevant.~~ - ~~[ ] All breaking changes documented.~~
## Description Some minor typos missed earlier. ## Breaking Changes <!-- Optional, if there are any breaking changes document them, including how to migrate older code. --> ## Notes & open questions Followup from n0-computer#2365 ## Change checklist - [x] Self-review. - [x] Documentation updates if relevant. - ~~[ ] Tests if relevant.~~ - ~~[ ] All breaking changes documented.~~
Description
This follows
https://github.com/rust-lang/rfcs/blob/master/text/1574-more-api-documentation-conventions.md#appendix-a-full-conventions-text
and updates the doc comments in an attempt to be more user-friendly.
Breaking Changes
Notes & open questions
Change checklist
[ ] Tests if relevant.[ ] All breaking changes documented.