Skip to content

add a rustfmt detail to CONTRIBUTING.md#5343

Merged
bors merged 1 commit intorust-lang:masterfrom
dwijnand:contributing-fmt
Apr 11, 2018
Merged

add a rustfmt detail to CONTRIBUTING.md#5343
bors merged 1 commit intorust-lang:masterfrom
dwijnand:contributing-fmt

Conversation

@dwijnand
Copy link
Contributor

stealing from the info in #5301 (comment)

@rust-highfive
Copy link

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@matklad
Copy link
Contributor

matklad commented Apr 11, 2018

Nice! Ideally, we should provide an example git hook for this. I’ve written one, but I’ve accidentally deleted the repository with it :(

@bors r+

@bors
Copy link
Contributor

bors commented Apr 11, 2018

📌 Commit 2f4a40b has been approved by matklad

@bors
Copy link
Contributor

bors commented Apr 11, 2018

⌛ Testing commit 2f4a40b with merge 4b8af3a...

bors added a commit that referenced this pull request Apr 11, 2018
add a rustfmt detail to CONTRIBUTING.md

stealing from the info in #5301 (comment)
@dwijnand
Copy link
Contributor Author

Ideally, we should provide an example git hook for this.

also ideally rustfmt had a flag to tell it to reformat only changed files (tracked as rust-lang/rustfmt#1324)

@bors
Copy link
Contributor

bors commented Apr 11, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 4b8af3a to master...

@bors bors merged commit 2f4a40b into rust-lang:master Apr 11, 2018
@dwijnand dwijnand deleted the contributing-fmt branch April 11, 2018 12:26
@ehuss ehuss added this to the 1.27.0 milestone Feb 6, 2022
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.

6 participants