-
Notifications
You must be signed in to change notification settings - Fork 2.6k
docs: move docs building process to contributor guide #15854
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: master
Are you sure you want to change the base?
Conversation
cc @zdivelbiss if you have any other suggestion to this guide. |
The templates are located in | ||
<https://github.com/rust-lang/cargo/tree/master/src/doc/man> | ||
and are converted into three formats: | ||
|
||
The templates are located in <https://github.com/rust-lang/cargo/tree/master/src/doc/man> | ||
and are converted into three formats: |
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.
These lines are duplicated (probably unintentionally?).
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.
Nice catch. Fixed!
This looks great! I hadn't known this existed (I have a keen eye when it comes to subconsciously skipping over important documents lol). Hopefully adding it to the contribution guide will help others, too. |
1. Troff man pages --- used by the `cargo help` command, | ||
saved in <https://github.com/rust-lang/cargo/tree/master/src/etc/man>. |
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.
These are also used by distributions to provide man pages which they can install. In the pre-built versions we provide, they are located in $SYSROOT/share/man/man1
. (Controlled with the install.mandir
bootstrap config option.)
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.
Updated. Let me know if it is still not good enough.
What does this PR try to resolve?
Pull https://github.com/rust-lang/cargo/blob/71eb84f21aef43c07580c6aed6f806a6299f5042/src/doc/README.md into our contributor guide for visibility.
Also a writing guideline is added to the chapter, as well as some minor polish.
This is created after seeing the contributing frictions in #15838 (comment).
How to test and review this PR?