Skip to content

Conversation

richo
Copy link
Contributor

@richo richo commented Mar 30, 2015

I also wanted to unignore https://github.com/rust-lang/rust/blob/master/src/libsyntax/ext/expand.rs#L1768-L1777 since the issue it references is closed, but the test fails, and it's internals aren't super clear to me.

@rust-highfive
Copy link
Contributor

r? @sfackler

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

@bors
Copy link
Collaborator

bors commented Apr 1, 2015

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

richo added 2 commits April 1, 2015 11:04
This issue is long closed. It seems that unwrapping will always panic
though.
@richo
Copy link
Contributor Author

richo commented Apr 1, 2015

Rebased.

@Manishearth
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 2, 2015

📌 Commit 85c74c6 has been approved by Manishearth

@bors
Copy link
Collaborator

bors commented Apr 2, 2015

⌛ Testing commit 85c74c6 with merge edac3ce...

bors added a commit that referenced this pull request Apr 2, 2015
I also wanted to unignore https://github.com/rust-lang/rust/blob/master/src/libsyntax/ext/expand.rs#L1768-L1777 since the issue it references is closed, but the test fails, and it's internals aren't super clear to me.
@bors
Copy link
Collaborator

bors commented Apr 2, 2015

@bors bors merged commit 85c74c6 into rust-lang:master Apr 2, 2015
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.

5 participants