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.

Sign up for GitHub

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

Jump to bottom

rustfmt libgraphviz #28255

Merged
merged 2 commits into from
Sep 11, 2015
Merged

rustfmt libgraphviz #28255

merged 2 commits into from
Sep 11, 2015

Conversation

nrc
Copy link
Member

@nrc nrc commented Sep 5, 2015

@bors
Copy link
Contributor

bors commented Sep 6, 2015

☔ The latest upstream changes (presumably #27893) made this pull request unmergeable. Please resolve the merge conflicts.

@pnkfelix
Copy link
Member

looks fine, r=me

@nrc
Copy link
Member Author

nrc commented Sep 11, 2015

@bors r=pnkfelix

@bors
Copy link
Contributor

bors commented Sep 11, 2015

📌 Commit 309ee93 has been approved by pnkfelix

@bors
Copy link
Contributor

bors commented Sep 11, 2015

⌛ Testing commit 309ee93 with merge 883b5cf...

bors added a commit that referenced this pull request Sep 11, 2015
@bors
Auto merge of #28255 - nrc:fmt4, r=pnkfelix
883b5cf 
r? @pnkfelix
@bors
Copy link
Contributor

bors commented Sep 11, 2015

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt

@bors bors merged commit 309ee93 into rust-lang:master Sep 11, 2015
@drrosa
Copy link

drrosa commented Sep 12, 2015

@nrc If different from small_tabs.toml, would you mind sharing the rustfmt.toml you are using here in the rust-lang repo?

@nrc
Copy link
Member Author

nrc commented Sep 13, 2015

@rosatamsen no rustfmt.toml, just using the defaults

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pnkfelix pnkfelix

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nrc @bors @pnkfelix @drrosa