-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Refactor user-guide of book
#529
Conversation
7f1c448
to
3a1b78a
Compare
* `setup`: Deduplicate setup * `setup-app`: name `cargo.toml`; add `defmt-itm` * `macros`: Language * `primitives`: Table-ize and drop `u24` (got dropped in (#521)) * `str`: Add links to doc-rs * ...
4493ec5
to
37d05b9
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.
I love the visual changes with the use of blockquotes and emoji. 👍
I left several inline comments but otherwise this looks good to me. Feel free to send this to bors after addressing my comments / suggestions
When the alternate form is used for hex and binary, the `0x`/`0b` length is subtracted from the | ||
leading zeros. This matches [`core::fmt` behavior](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=b11809759f975e266251f7968e542756). No further | ||
customization is supported (at the moment). | ||
## Alternate printing |
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.
could you include in this section that {}, "hello"
is formatted as hello
but that {:#}, "hello"
is formatted as "hello"
and note that this matches core::fmt
behavior
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.
I guess you mean {:?}
? I am generally unsure what the ?
does anyways, since we don't have separate Debug
formatting.
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.
I'll merge it without this being resolved and will raise the question in zulip :)
bors r+ |
Build succeeded: |
This PR aims to update the user guide of the book to the dirbaio PRs and is doing quite some refactoring on the side.
Preview: https://deploy-preview-529--admiring-dubinsky-56dff5.netlify.app