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

Rustup #2889

Merged
merged 7 commits into from
Jul 7, 2018
Merged

Rustup #2889

merged 7 commits into from
Jul 7, 2018

Conversation

oli-obk
Copy link
Contributor

@oli-obk oli-obk commented Jul 1, 2018

No description provided.

@mati865
Copy link
Contributor

mati865 commented Jul 3, 2018

Probably good time to revive #2551
Something like https://github.com/rust-lang/rust/pull/52004/files#diff-354f30a63fb0907d4ad57269548329e3R172 could be used to make it run only on rustup branch

@oli-obk
Copy link
Contributor Author

oli-obk commented Jul 3, 2018

Good idea. But it's not that important. This PR exists just so I can point to it from the rustc repo. Once all that hits nightly I can just press the merge button

@mati865
Copy link
Contributor

mati865 commented Jul 3, 2018

Should have worded it better.
Future PRs like this one could greatly benefit from working CI.

@ghost ghost mentioned this pull request Jul 7, 2018
@Manishearth Manishearth merged commit aad450e into master Jul 7, 2018
@Manishearth Manishearth deleted the rustup branch July 8, 2018 07:06
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