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

Review fixes for #32878 #32880

Merged
merged 2 commits into from
Apr 11, 2016
Merged

Review fixes for #32878 #32880

merged 2 commits into from
Apr 11, 2016

Conversation

Manishearth
Copy link
Member

#32878 was accidentally committed directly to master due to use of the GH interface edit button.

This contains review fixes for the PR.

@Manishearth
Copy link
Member Author

@bors delegate=GuillaumeGomez

@bors
Copy link
Contributor

bors commented Apr 11, 2016

✌️ @GuillaumeGomez can now approve this pull request


```compile_fail
#![feature(specialization)]

trait SpaceLama {
trait SpaceLlama {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why SpaceLlama?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Llama is the animal, Lama is the Buddhist monk 😄

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is how rust improves every day 🌞

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bluss: Well, I invite you to take a look at my previous error explanations/test cases PRs then. :p

@GuillaumeGomez
Copy link
Member

@bors: r+ p=10 force

@bors
Copy link
Contributor

bors commented Apr 11, 2016

📌 Commit 69095bb has been approved by GuillaumeGomez

@bors
Copy link
Contributor

bors commented Apr 11, 2016

⌛ Testing commit 69095bb with merge 87e2eb1...

bors added a commit that referenced this pull request Apr 11, 2016
Review fixes for #32878

This contains review fixes for the PR.
@Manishearth
Copy link
Member Author

(Moving it up in the queue because homu breaks on direct pushes anyway, so best get this out of the way)

@bors bors merged commit 69095bb into rust-lang:master Apr 11, 2016
@Manishearth Manishearth deleted the fix branch December 2, 2016 19:03
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