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

Clean code a bit #36048

Merged
merged 1 commit into from
Sep 8, 2016
Merged

Clean code a bit #36048

merged 1 commit into from
Sep 8, 2016

Conversation

GuillaumeGomez
Copy link
Member

No description provided.

@rust-highfive
Copy link
Collaborator

r? @aturon

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

@GuillaumeGomez
Copy link
Member Author

r? @jonathandturner

@rust-highfive rust-highfive assigned sophiajt and unassigned aturon Aug 30, 2016
@sophiajt
Copy link
Contributor

I don't do much in the std lib. Assigning back to aturon, who might know a good reviewer.

r? @aturon

@rust-highfive rust-highfive assigned aturon and unassigned sophiajt Aug 30, 2016
@GuillaumeGomez
Copy link
Member Author

cc @brson @alexcrichton or whoever who can review this PR please. :)

@brson
Copy link
Contributor

brson commented Sep 7, 2016

@bors r+. Wonder where that loop came from...

@GuillaumeGomez
Copy link
Member Author

@brson: You failed the r+ I think. :p

@bors: r=brson

@bors
Copy link
Contributor

bors commented Sep 8, 2016

📌 Commit 286ae72 has been approved by brson

@bors
Copy link
Contributor

bors commented Sep 8, 2016

⌛ Testing commit 286ae72 with merge c615b21...

bors added a commit that referenced this pull request Sep 8, 2016
@bors bors merged commit 286ae72 into rust-lang:master Sep 8, 2016
@GuillaumeGomez GuillaumeGomez deleted the code_clean branch September 9, 2016 09:40
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