-
Notifications
You must be signed in to change notification settings - Fork 483
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
Items chapter cleanup and improvements #91
Conversation
* Make the list at the start of the chapter be in the same order as the actual chapter * Say some more about C like enums * Make the wrapping more consistent * Fix some header levels * Make more of the code examples tested * Be a bit clearer on RFC 116
src/items.md
Outdated
[path]: paths.html | ||
[modules]: #modules | ||
[blocks]: ../grammar.html#block-expressions | ||
[path]: paths.html [modules]: #modules [blocks]: |
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.
Doesn't this break the links?
src/items.md
Outdated
|
||
[struct type]: types.html#struct-types | ||
[tuple type]: types.html#tuple-types | ||
[struct type]: types.html#struct-types [tuple type]: types.html#tuple-types |
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.
Same here with does this actually break links? We probably want them on multiple lines even if they don't.
LGTM minus the two comments. |
88b90fe
to
2f04efa
Compare
2f04efa
to
7c8ff20
Compare
This LGTM too; a small rebase is needed due to a grammar PR; ping me when you've rebased and I will merge! |
@steveklabnik this should now be good to go. |
Thank you! |
This PR makes (mostly) small improvements to the items chapter