Skip to content

Conversation

@Manishearth
Copy link
Member

not 100% sure of the Symbol stuff

r? @oli-obk

@phansch
Copy link
Contributor

phansch commented May 22, 2019

Might be worth to disable the cargo integration test until #4121 is fixed so we can get this merged.

@Manishearth
Copy link
Member Author

Going to bed soon, feel free to push that on my commits and merge 😄

@oli-obk
Copy link
Contributor

oli-obk commented May 22, 2019

@bors r+

@bors
Copy link
Contributor

bors commented May 22, 2019

📌 Commit 833ef00 has been approved by oli-obk

@bors
Copy link
Contributor

bors commented May 22, 2019

⌛ Testing commit 833ef00 with merge a8eeb7c...

bors added a commit that referenced this pull request May 22, 2019
Rustup to rustc 1.36.0-nightly (dbfe70d 2019-05-22)

not 100% sure of the Symbol stuff

r? @oli-obk
@bors
Copy link
Contributor

bors commented May 22, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: oli-obk
Pushing a8eeb7c to master...

@bors bors merged commit 833ef00 into rust-lang:master May 22, 2019
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.

4 participants