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

Updated CI config to reflect deprecation of rustup uninstall #1111

Merged
merged 1 commit into from
Dec 8, 2019

Conversation

MarkusPettersson98
Copy link
Contributor

In the same spirit as #1110 😄

This PR from the rustup repository brought me here: rust-lang/rustup#2148

TL;DR With rustup 1.21.0 rustup install and rustup uninstall are being deprecated in favor of rustup toolchain install and rustup toolchain uninstall. There's plenty of code/documentation out there that needs to be updated to reflect this.

Thought It would be cool to help, however small the change may be. :)

Keep up the great work!

@RalfJung
Copy link
Member

RalfJung commented Dec 8, 2019

Looking good, thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Dec 8, 2019

📌 Commit c9e7fe8 has been approved by RalfJung

bors added a commit that referenced this pull request Dec 8, 2019
Updated CI config to reflect deprecation of rustup uninstall

In the same spirit as #1110 😄

This PR from the rustup repository brought me here: rust-lang/rustup#2148

TL;DR With rustup 1.21.0 `rustup install` and `rustup uninstall` are being deprecated in favor of `rustup toolchain install` and `rustup toolchain uninstall`. There's plenty of code/documentation out there that needs to be updated to reflect this.

Thought It would be cool to help, however small the change may be. :)

Keep up the great work!
@bors
Copy link
Contributor

bors commented Dec 8, 2019

⌛ Testing commit c9e7fe8 with merge 4ebc030...

@bors
Copy link
Contributor

bors commented Dec 8, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 4ebc030 to master...

@bors bors merged commit c9e7fe8 into rust-lang:master Dec 8, 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.

3 participants