Skip to content

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

Merged
merged 8 commits into from
Sep 7, 2018
Merged

Conversation

Havvy
Copy link
Contributor

@Havvy Havvy commented Aug 24, 2018

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!

@Havvy Havvy force-pushed the rustconf-rustfest branch from 0c4db03 to 4abeeb9 Compare August 24, 2018 05:18
@Havvy Havvy requested a review from steveklabnik August 24, 2018 05:22
Copy link
Contributor

@matthewjasper matthewjasper left a 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.

```

* The grammar and lexical structure is in blockquotes with either "Lexer" or
"Syntax" in **bold** as the first line in <sup>superscript</sup>.
Copy link
Contributor

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..."


* 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**.
Copy link
Contributor

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.

<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,
Copy link
Contributor

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.

@Havvy
Copy link
Contributor Author

Havvy commented Aug 25, 2018

All addressed.

@Havvy Havvy added the Language Cleanup Improvements to existing language which is correct but not clear, or missing examples, or the like. label Aug 29, 2018
[linkage]: linkage.html
[rustc book]: ../rustc/index.html
[undocumented]: undocumeted.html
Copy link
Contributor

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.
@Havvy Havvy force-pushed the rustconf-rustfest branch from e4aa57c to 0d7a062 Compare September 2, 2018 06:38
Copy link
Member

@steveklabnik steveklabnik left a 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

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
Copy link
Member

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'

@steveklabnik steveklabnik merged commit 40ebbee into rust-lang:master Sep 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Cleanup Improvements to existing language which is correct but not clear, or missing examples, or the like.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants