Skip to content
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

Reformat the entire codebase using cargo fmt #1585

Merged
merged 1 commit into from
Dec 26, 2018

Conversation

kinnison
Copy link
Contributor

To make sure that people using editors which use rustfmt by default
on saving don't touch more lines than intended when working on the
rustup codebase, we'll take the hit to format it all once.

Please note, this depends on #1583 (as it effectively contains that commit also)

@nrc
Copy link
Member

nrc commented Dec 18, 2018

Did you do any manual fixups or is this all just running cargo fmt?

@kinnison
Copy link
Contributor Author

kinnison commented Dec 19, 2018

@nrc The formatting commit was pure cargo fmt and no manual changes.

To make sure that people using editors which use rustfmt by default
on saving don't touch more lines than intended when working on the
rustup codebase, we'll take the hit to format it all once.
@kinnison
Copy link
Contributor Author

All I did was rebase against current master and confirm there were no further formatting changes needed.

Copy link
Member

@nrc nrc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@nrc nrc merged commit 61f57bb into rust-lang:master Dec 26, 2018
@kinnison kinnison deleted the kinnison/cargo-fmt branch January 9, 2019 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants