-
Notifications
You must be signed in to change notification settings - Fork 6
mess with the docs #1062
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
base: main
Are you sure you want to change the base?
mess with the docs #1062
Changes from 5 commits
4dc11b7
c565c8b
6b591ab
6a6800a
eed1bbf
308ec76
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,6 +3,7 @@ | |
|
|
||
| #![deny(clippy::all, clippy::pedantic)] | ||
| #![deny(rustdoc::all)] | ||
| #![allow(rustdoc::broken_intra_doc_links)] | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd rather not ignore these reports, could we fix the docs in I took a quick look and the only one I got to fix was: diff --git i/nat/src/stateful/apalloc/setup.rs w/nat/src/stateful/apalloc/setup.rs
index 71508eb41328..f83f525c44ad 100644
--- i/nat/src/stateful/apalloc/setup.rs
+++ w/nat/src/stateful/apalloc/setup.rs
@@ -19,8 +19,9 @@ use std::time::Duration;
use tracing::debug;
impl NatDefaultAllocator {
- /// Build a [`NatDefaultAllocator`] from information collected from a [`VpcTable`] object. This
- /// information is passed as a [`StatefulNatConfig`].
+ /// Build a [`NatDefaultAllocator`] from information collected from a
+ /// [`VpcTable`][config::external::overlay::vpc::VpcTable] object.
+ /// This information is passed as a [`StatefulNatConfig`].
///
/// # Returns
///Let me know if you got more, if I can reproduce I'm happy to address them in a separate PR.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think there were very few. Happy if we fix the docs in each crate, more than brutally disabling the lints as I did. |
||
| #![allow(clippy::missing_errors_doc)] | ||
|
|
||
| //! Network Address Translation (NAT) for the dataplane | ||
|
|
||
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.
❤️