-
Notifications
You must be signed in to change notification settings - Fork 645
Documentation #215
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
Documentation #215
Conversation
otherwise "cargo doc" fails to work
🎊 awesome! |
@@ -18,6 +18,7 @@ use std::io::prelude::*; | |||
|
|||
use cargo_registry::Crate; | |||
|
|||
#[allow(dead_code)] | |||
fn main() { |
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.
iirc the "proper" way to do this is #[cfg(not(test))]
. May be misremembering for doc
.
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.
if I replace this with #[cfg(not(test))]
I'm getting the following sort of errors again:
src/bin/delete-version.rs:22:1: 31:2 error: function is never used: `main`, #[deny(dead_code)] on by default
No opinion on the middleware changes. (I also don't have review rights anyway, so y'all'll need to wait on someone else for that) |
Thanks! |
This PR adds documentation comments to the
user
module.