-
Notifications
You must be signed in to change notification settings - Fork 895
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
Print new version while rustup self update
#542
Comments
I've been indecisive about this. I originally didn't do this because I wanted the rustup version to not be important - it's just continually updated. And with These are the outputs for the two types of updates today
Maybe the rustup version could be put on the info log line that mentions the update: "rustup successfully updated to X.Y.Z" . That way with |
"rustup successfully updated to X.Y.Z" is nice to have. |
OK, let's change the "rustup updated successfully" message to "rustup successfully updated to X.Y.Z". Getting the version will require running the newly-downloaded bin. |
Hi I'd like to try to tackle this issue if nobody is already working on it. |
@julienXX You got it! |
@julienXX How's it going? |
@brson sorry been very busy these days but I still intend to submit a PR hopefully in the next few days. |
Thanks @julienXX ! I added a comment on the PR. |
before or after update complete.
The text was updated successfully, but these errors were encountered: