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 README to reflect deprecation of rustup install #1110

Merged
merged 1 commit into from
Dec 8, 2019

Conversation

MarkusPettersson98
Copy link
Contributor

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

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Dec 8, 2019

📌 Commit ef37a4c has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Dec 8, 2019

⌛ Testing commit ef37a4c with merge ea59d6a...

bors added a commit that referenced this pull request Dec 8, 2019
Updated README to reflect deprecation of rustup install

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

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

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