-
Notifications
You must be signed in to change notification settings - Fork 13.7k
book: some Crates and Modules nits #23401
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
r? @pcwalton (rust_highfive has picked a reviewer for you, use r? to override) |
@@ -241,15 +221,9 @@ fn hello() -> String { | |||
} | |||
``` | |||
|
|||
Of course, you can copy and paste this from this web page, or just type | |||
something else. It's not important that you actually put "konnichiwa" to learn | |||
about the module system. |
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.
why remove this?
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.
Sounds obvious, so doesn't add value.
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's important because of the unicode, and also, it's just style.
r=me after re-adding the idiom bit |
@bors: r+ rollup |
📌 Commit 8a500de has been approved by |
Conflicts: src/doc/trpl/crates-and-modules.md
- Successful merges: rust-lang#22954, rust-lang#23119, rust-lang#23509, rust-lang#23561, rust-lang#23590, rust-lang#23607, rust-lang#23608, rust-lang#23618, rust-lang#23622, rust-lang#23639, rust-lang#23641 - Failed merges: rust-lang#23401
No description provided.