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

rewrite the method-receiver matching code #30582

Merged
merged 2 commits into from
Dec 28, 2015
Merged

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Dec 27, 2015

the old code was so terrible.

r? @eddyb

the old code was *so terrible*.
@eddyb
Copy link
Member

eddyb commented Dec 27, 2015

@bors r+

@bors
Copy link
Contributor

bors commented Dec 27, 2015

📌 Commit efc4575 has been approved by eddyb

@bors
Copy link
Contributor

bors commented Dec 27, 2015

⌛ Testing commit efc4575 with merge 54ac583...

I forgot about it in the previous commit
@arielb1
Copy link
Contributor Author

arielb1 commented Dec 27, 2015

@eddyb could you re-r+ the new commit?

@eddyb
Copy link
Member

eddyb commented Dec 27, 2015

@bors r+ c00b5dc

Although I don't know how to stop the previous build.
cc @Manishearth

@bors
Copy link
Contributor

bors commented Dec 27, 2015

⌛ Testing commit c00b5dc with merge b4707eb...

bors added a commit that referenced this pull request Dec 27, 2015
the old code was *so terrible*.

r? @eddyb
@arielb1
Copy link
Contributor Author

arielb1 commented Dec 27, 2015

I already stopped it.

1 similar comment
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.

3 participants