Skip to content

Support /proc/net/snmp and /proc/net/snmp6#281

Merged
eminence merged 4 commits intoeminence:masterfrom
yukiiiteru:net-snmp
Sep 23, 2023
Merged

Support /proc/net/snmp and /proc/net/snmp6#281
eminence merged 4 commits intoeminence:masterfrom
yukiiiteru:net-snmp

Conversation

@yukiiiteru
Copy link
Contributor

No description provided.

@eminence
Copy link
Owner

Just did a quick skim, and this looks great, thank you. I am especially impressed with all of the nice docs. Can you take a look at the CI failure, and also update support.md ?

warning: lint `broken_intra_doc_links` has been renamed to `rustdoc::broken_intra_doc_links`
 --> procfs/src/lib.rs:4:9
  |
4 | #![deny(broken_intra_doc_links, invalid_html_tags)]
  |         ^^^^^^^^^^^^^^^^^^^^^^ help: use the new name: `rustdoc::broken_intra_doc_links`
  |
  = note: `#[warn(renamed_and_removed_lints)]` on by default

warning: lint `invalid_html_tags` has been renamed to `rustdoc::invalid_html_tags`
 --> procfs/src/lib.rs:4:33
  |
4 | #![deny(broken_intra_doc_links, invalid_html_tags)]
  |                                 ^^^^^^^^^^^^^^^^^ help: use the new name: `rustdoc::invalid_html_tags`
@yukiiiteru
Copy link
Contributor Author

Just did a quick skim, and this looks great, thank you. I am especially impressed with all of the nice docs. Can you take a look at the CI failure, and also update support.md ?

Sorry I overlooked the CI failure earlier. An update has been pushed, please review it.

BTW, most of the docs are from the official documentation, e.g., RFC1213 and SNMP counter. I just copied them, but some non-rfc fields do not have any doc 😢

@eminence eminence merged commit a9b5239 into eminence:master Sep 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants