Uh oh!
There was an error while loading. Please reload this page.
Fixing basic typos in Doc Comments - #42812
Conversation
rust-highfive
commented
Jun 21, 2017
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @pnkfelix (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
Mark-Simulacrum
commented
Jun 21, 2017
Thanks! @bors r+ rollup |
bors
commented
Jun 21, 2017
📌 Commit a21e056 has been approved by |
| /// | ||
| /// panic::set_hook(Box::new(|panic_info| { | ||
| /// if let Some(location) = panic_info.location() { | ||
| /// println!("panic occured in file '{}' at line {}", location.file(), location.line()); |
There was a problem hiding this comment.
there is now a tidy violation on this line, it needs to be wrapped
tidy check (x86_64-unknown-linux-gnu)
tidy error: /checkout/src/libstd/panicking.rs:224: line longer than 100 chars
There was a problem hiding this comment.
Now there's a new tidy error:
tidy error: /checkout/src/libstd/panicking.rs:224: trailing whitespace
You're going to need to remove that space at the end of this line
frewsxcv
commented
Jun 21, 2017
@bors r- |
| /// panic::set_hook(Box::new(|panic_info| { | ||
| /// if let Some(location) = panic_info.location() { | ||
| /// println!("panic occurred in file '{}' at line {}", location.file(), location.line()); | ||
| /// println!("panic occurred in file '{}' at line {}", location.file(), location.line()); |
There was a problem hiding this comment.
By convention, Rust uses four spaces for indentation, you'll need to do something like this instead:
/// println!("panic occurred in file '{}' at line {}", /// location.file(), location.line());frewsxcv
commented
Jun 22, 2017
Thanks! @bors r=Mark-Simulacrum |
bors
commented
Jun 22, 2017
📌 Commit 7ec4d14 has been approved by |
Fixing basic typos in Doc Comments Files checked : *.rs files inside 'src' directory. Only Doc comments (/// and //!) are fixed. Note: the non American-English spellings are left unchanged.
frewsxcv
commented
Jun 22, 2017
@bors r- |
steveklabnik
commented
Jun 22, 2017
We specifically do use |
steveklabnik
commented
Jun 23, 2017
Traivis is failing due to After travis passes and the commits are squashed, r=me! |
bors
commented
Jun 30, 2017
☔ The latest upstream changes (presumably #42924) made this pull request unmergeable. Please resolve the merge conflicts. |
aidanhs
commented
Jul 5, 2017
Hi @Fourchaux, have you had a chance to look at the trailing whitespace issue raised by the tidy check? |
aidanhs
commented
Jul 5, 2017
Oh and it'd be great if you could squash your commits into one while you're at it! |
aidanhs
commented
Jul 13, 2017
Hi @Fourchaux, as this has been inactive for a while I'm going to it to keep the PR queue tidy - if you do get round to updating, feel free to reopen. Thanks again for the changes! |
Fix typos & us spellings Fixing some typos and non en-US spellings. (Update of PR rust-lang#42812 )
Fix typos & us spellings Fixing some typos and non en-US spellings. (Update of PR rust-lang/rust#42812 )
Files checked : *.rs files inside 'src' directory.
Only Doc comments (/// and //!) are fixed.
Note: the non American-English spellings are left unchanged.