-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Add let keyword doc #53231
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
Add let keyword doc #53231
Conversation
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
c3fe769
to
e2a92a2
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
e2a92a2
to
36f6884
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
36f6884
to
7a87481
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
7a87481
to
a71aa1a
Compare
How will this appear in the doc and search? (I hope it won't say "let (primitive)" 😄) |
src/libstd/keyword_docs.rs
Outdated
/// | ||
/// ```rust | ||
/// # #![allow(unused_assignments)] fn main() { | ||
/// let mut x = 3; // We create a mutable variable named `x` with the value `3`. |
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.
You don't have to include a fn main
to apply these crate-level lints. Rustdoc will already move them outside its generated main
function.
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.
Oh cool! Completely forgot about it. :)
a71aa1a
to
f9f934f
Compare
Updated. |
Looks good to me, thanks! @bors r+ rollup |
📌 Commit f9f934f has been approved by |
🌲 The tree is currently closed for pull requests below priority 10, this pull request will be tested once the tree is reopened |
…Misdreavus Add let keyword doc Part of rust-lang#34601.
…Misdreavus Add let keyword doc Part of rust-lang#34601. r? @rust-lang/docs
Rollup of 15 pull requests Successful merges: - #52955 (Update compiler test documentation) - #53019 (Don't collect() when size_hint is useless) - #53025 (Consider changing assert! to debug_assert! when it calls visit_with) - #53059 (Remove explicit returns where unnecessary) - #53165 ( Add aarch64-unknown-netbsd target) - #53210 (Deny future duplication of rustc-ap-syntax) - #53223 (A few cleanups for rustc_data_structures) - #53230 ([nll] enable feature(nll) on various crates for bootstrap: part 4) - #53231 (Add let keyword doc) - #53240 (Add individual documentation for <integer>`.swap_bytes`/.`reverse_bits`) - #53253 (Remove unwanted console log) - #53264 (Show that Command can be reused and remodified) - #53267 (Fix styles) - #53273 (Add links to std::char::REPLACEMENT_CHARACTER from docs.) - #53283 (wherein we suggest float for integer literals where a float was expected) Failed merges: r? @ghost
Part of #34601.
r? @rust-lang/docs