-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 #3893
Rustup #3893
Conversation
As suggested by eddyb
@bors r+ |
📌 Commit 8cfc91b has been approved by |
Rustup Supersedes #3889 Addresses some review comments from previous PR and rustups to rust-lang/rust#58899
Formatting with @bors retry |
@mati865: 🔑 Insufficient privileges: not in try users |
@bors r+ |
📌 Commit 6cb0605 has been approved by |
Rustup Supersedes #3889 Addresses some review comments from previous PR and rustups to rust-lang/rust#58899
💔 Test failed - checks-travis |
@bors retry |
Rustup Supersedes #3889 Addresses some review comments from previous PR and rustups to rust-lang/rust#58899
💥 Test timed out |
@bors retry |
Rustup Supersedes #3889 Addresses some review comments from previous PR and rustups to rust-lang/rust#58899
Appveyor is broken 😞 |
@bors retry p=10 |
Rustup Supersedes #3889 Addresses some review comments from previous PR and rustups to rust-lang/rust#58899
Please let me know if/when you decide to update tools in the Rust tree - I'd like to update to the current RLS master (which also means pulling in Rustfmt 1.1) in tandem with this. |
@Xanewok you can upgrade all the tools in Rust repo at same time if you want. |
☀️ Test successful - checks-travis, status-appveyor |
Supersedes #3889
Addresses some review comments from previous PR and rustups to rust-lang/rust#58899