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

ENH: Add release notes to release #13191

Closed
corneliusroemer opened this issue Sep 5, 2022 · 7 comments · Fixed by #13771
Closed

ENH: Add release notes to release #13191

corneliusroemer opened this issue Sep 5, 2022 · 7 comments · Fixed by #13771
Assignees
Labels
A-infra CI and workflow issues C-feature Category: feature request good first issue S-actionable Someone could pick this issue up and work on it right now

Comments

@corneliusroemer
Copy link

corneliusroemer commented Sep 5, 2022

When having a look at the latest release I was surprised to find there were no release notes.

https://github.com/rust-lang/rust-analyzer/releases/tag/2022-09-05

This seems to be the rule - a but surprising for such a big project.

Would it be hard to populate the release notes with whatever is in the changelog?

I know the releases are a GitHub feature and not strictly necessary, but since releases get announced to all watchers it would be nice to have the info there.

It seems to exist already here: https://rust-analyzer.github.io/thisweek/2022/08/29/changelog-144.html

Screenshot_2022-09-05-09-15-54-22_320a9a695de7cdce83ed5281148d6f19.jpg

@lnicola
Copy link
Member

lnicola commented Sep 5, 2022

It's not very easy either. The release notes are generated automatically when the release is triggered, then pretty heavily edited. We might be able to insert a link to e.g. https://rust-analyzer.github.io/thisweek/2022/09/05/changelog-145.html, but even that can pose problems with out-of-band releases like https://rust-analyzer.github.io/thisweek/2022/05/17/changelog-129-5.html.

@lnicola lnicola added good first issue S-actionable Someone could pick this issue up and work on it right now C-feature Category: feature request A-infra CI and workflow issues labels Sep 5, 2022
@corneliusroemer
Copy link
Author

corneliusroemer commented Sep 5, 2022 via email

@lnicola
Copy link
Member

lnicola commented Sep 5, 2022

I mean, the GitHub release is published automatically at the moment. There's nothing marking it as "out of band", and I only gave those as a not-obvious example because they tend to break the URL and release numbering.

I'm not saying that this can't be done, but we'd have to change almost everything about how the releases are made.

@noritada
Copy link
Contributor

I would like to try to solve this issue.

I checked the following files and understood that GitHub Releases entries are generated in a GitHub Actions workflow along with release binaries and that templates for changelog entries are generated in the local environment.

The simplest solution would be to update the GitHub Releases entry when a new edited changelog entry is pushed to the website repository. Do you have any suggestions?

@noritada
Copy link
Contributor

@rustbot claim

@lnicola
Copy link
Member

lnicola commented Dec 4, 2022

Sorry for taking so long. I didn't realize that we could update GitHub releases after publishing them, but that's indeed possible: https://docs.github.com/en/rest/releases/releases?apiVersion=2022-11-28#update-a-release.

This could be an extra step, something like cargo xtask publish-release-notes, or it could even be done directly from GHA (with more work to match the updated file to the release). The first version might be easier for now, especially given that the original version is AsciiDoc, while GitHub Releases probably use Markdown. There might be some other feature gaps too, like image support.

@lnicola
Copy link
Member

lnicola commented Dec 26, 2022

As seen on GitHub 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-infra CI and workflow issues C-feature Category: feature request good first issue S-actionable Someone could pick this issue up and work on it right now
Projects
None yet
3 participants