Skip to content

Format libcore with rustfmt#66691

Merged
bors merged 2 commits intorust-lang:masterfrom
dtolnay:fmt0
Nov 27, 2019
Merged

Format libcore with rustfmt#66691
bors merged 2 commits intorust-lang:masterfrom
dtolnay:fmt0

Conversation

@dtolnay
Copy link
Copy Markdown
Member

@dtolnay dtolnay commented Nov 24, 2019

I am interested in whether we can begin cautious incremental progress on #66688 and assess along the way whether we can keep the disruption sufficiently small.

This PR applies rustfmt with default settings to files in src/libcore that are not involved in any currently open PR to minimize merge conflicts. The list of files involved in open PRs was determined by querying GitHub's GraphQL API with this script.

With the list of files from the script in outstanding_files, the relevant commands were:

$ find src/libcore -name '*.rs' | xargs rustfmt --edition=2018
$ rg libcore outstanding_files | xargs git checkout --

Repeating this process several months apart should get us coverage of most of the rest of libcore.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-cleanup Category: PRs that clean code up or issues documenting cleanup. merged-by-bors This PR was explicitly merged by bors. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants