-
Notifications
You must be signed in to change notification settings - Fork 893
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
Prepare 1.28.0 release #4079
base: master
Are you sure you want to change the base?
Prepare 1.28.0 release #4079
Conversation
Also Line 36 in c8eddcb
|
@ChrisDenton Thanks for pointing it out! That's part of the plan, yeah, but it cannot be done in this PR for technical reasons. (It should be done right after this PR though.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a few points I'd like to add, but this is looking wonderful. Many thanks!
Do you mean the addition of Chris to the team? If there are other things you'd like to add, now would be a good time to mention that, I think? |
adb19a5
to
07bdcf7
Compare
Addressed the issues you've mentioned so far! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
|
||
- The default download backend has been changed from reqwest with native-tls to reqwest with rustls. [pr#3798] | ||
- `RUSTUP_USE_CURL` and `RUSTUP_USE_RUSTLS` can still be used to change the download backend | ||
if the new backend causes issues. If issues do happen, please [let us know](https://github.com/rust-lang/rustup/issues/3806). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might not be a significant point. However, we(at least recent three releases) usually prefer placing all links at the end of the document. For example:
[let us know]: #3806
I simply followed the format used in previous changelog sections. That said, I think it's acceptable to have two different styles coexisting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
placing all links at the end of the document
I second this, despite the fact that this PR tries to maximally utilize the GitHub-generated detailed changelog. For the "manual" part, at least, it'd still be better to simply follow the established convention.
@djc That has been all. Thanks a lot! If anything happens, we can still modify the changelog during the beta testing period :) |
This is the 1st PR for the
1.28.0
release according to our release process.Checklist
Draft the release blog post(Not required, since we're doing a beta release).