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

run rustfmt on libstd/collections/hash folder #36753

Merged
merged 1 commit into from
Oct 7, 2016
Merged

run rustfmt on libstd/collections/hash folder #36753

merged 1 commit into from
Oct 7, 2016

Conversation

srinivasreddy
Copy link
Contributor

No description provided.

@rust-highfive
Copy link
Collaborator

r? @brson

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

@bluss
Copy link
Member

bluss commented Sep 27, 2016

There are two hashmap PRs open, might not be the best time? That said I don't know how much this will disrupt.

@srinivasreddy
Copy link
Contributor Author

srinivasreddy commented Sep 27, 2016

@bluss Understood. I concede. 👍 Will re-open when that PR is merged.

@srinivasreddy
Copy link
Contributor Author

r? @nrc What do you think of this? This PR looks clean to me.

@rust-highfive rust-highfive assigned nrc and unassigned brson Oct 5, 2016
@nrc
Copy link
Member

nrc commented Oct 6, 2016

Looks good, thanks @srinivasreddy

@bors: r+

@bors
Copy link
Contributor

bors commented Oct 6, 2016

📌 Commit 45f2b6a has been approved by nrc

@bors
Copy link
Contributor

bors commented Oct 7, 2016

⌛ Testing commit 45f2b6a with merge b4e8972...

bors added a commit that referenced this pull request Oct 7, 2016
run rustfmt on libstd/collections/hash folder
@bors bors merged commit 45f2b6a into rust-lang:master Oct 7, 2016
@srinivasreddy srinivasreddy deleted the hash branch August 24, 2019 06:26
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