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

Prepare 1.28.0 release #4079

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Prepare 1.28.0 release #4079

wants to merge 2 commits into from

Conversation

djc
Copy link
Contributor

@djc djc commented Nov 15, 2024

This is the 1st PR for the 1.28.0 release according to our release process.

Checklist

  • Bump version in the manifest.
  • Update the changelog.
  • Draft the release blog post (Not required, since we're doing a beta release).

CHANGELOG.md Show resolved Hide resolved
@ChrisDenton
Copy link
Member

Also

rustup-init 1.27.1 (a8e4f5c64 2024-04-24)
?

@rami3l rami3l added this to the 1.28.0 milestone Nov 15, 2024
CHANGELOG.md Outdated Show resolved Hide resolved
@rami3l
Copy link
Member

rami3l commented Nov 15, 2024

Also

rustup-init 1.27.1 (a8e4f5c64 2024-04-24)

?

@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.)

Copy link
Member

@rami3l rami3l left a 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!

CHANGELOG.md Outdated Show resolved Hide resolved
@djc
Copy link
Contributor Author

djc commented Nov 19, 2024

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?

@djc djc force-pushed the prepare-1.28.0 branch 2 times, most recently from adb19a5 to 07bdcf7 Compare November 19, 2024 10:21
@djc
Copy link
Contributor Author

djc commented Nov 19, 2024

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?

Addressed the issues you've mentioned so far!

Copy link
Member

@Rustin170506 Rustin170506 left a 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).
Copy link
Member

@Rustin170506 Rustin170506 Nov 19, 2024

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.

Copy link
Member

@rami3l rami3l Nov 19, 2024

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.

@rami3l
Copy link
Member

rami3l commented Nov 19, 2024

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?

Addressed the issues you've mentioned so far!

@djc That has been all. Thanks a lot! If anything happens, we can still modify the changelog during the beta testing period :)

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.

4 participants