-
Notifications
You must be signed in to change notification settings - Fork 533
Improve Introduction #408
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
Improve Introduction #408
Conversation
0c4db03
to
4abeeb9
Compare
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 looks great. There are a few small things that I noticed while reading this.
src/introduction.md
Outdated
``` | ||
|
||
* The grammar and lexical structure is in blockquotes with either "Lexer" or | ||
"Syntax" in **bold** as the first line in <sup>superscript</sup>. |
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.
Should this be "in bold superscript..."
src/introduction.md
Outdated
|
||
* Notes that contain useful information about the state of the book or point out | ||
useful, but mostly out of scope, information are in blockquotes that start with | ||
the word "Note:" in **bold**. |
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.
The example doesn't match this.
src/introduction.md
Outdated
<div class="warning"> | ||
|
||
Warning: This book is incomplete. Documenting everything takes a | ||
while. We have a [big issue] to track documentation for every Rust feature, |
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 issue was closed and replaced with smaller issues.
All addressed. |
[linkage]: linkage.html | ||
[rustc book]: ../rustc/index.html | ||
[undocumented]: undocumeted.html |
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.
It might be helpful to add a link to the Rustonomicon.
* Added another paragraph explaining that we don't document tools. * Expanded the contribution section. * Remove the part saying we may document some unstable stuff. * Added a how-to-read paragraph.
This involved moving paragraphs around as well.
e4aa57c
to
0d7a062
Compare
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 looks great! I have one typo fix and then this is good to go as far as I"m concerned
src/introduction.md
Outdated
Rust. For unstable features being worked on, see the [Unstable Book]. This was | ||
a recent change in scope, so unstable features are still documented, but are | ||
in the process of being removed. | ||
Similarly, this book does not usually book the specifics of `rustc` as a |
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.
the second 'book' here should be 'document'
Fixes #406
I totes stole the section title "How to Use This Book" from TRPL. ❤️
This PR will be easier to read by the individual commit.
Due to rendering being important, I have updated my website to render to this commit. Rendered!