-
Notifications
You must be signed in to change notification settings - Fork 537
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
Create release with template and release notes #22622
Conversation
ec6499f
to
b0da8e1
Compare
Won't recreate existing tag https://github.com/mozilla/addons-server/actions/runs/10635520149/job/29485287048 |
b0da8e1
to
0eae09a
Compare
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.
Some things I'm not clear on:
- how we're going to handle addons-frontend releases - will we just manually add the diff link in here, or something else?
- How the generated full changelog can work when there is an existing release note, with potentially important information that can't be overwritten with an entirely freshly generated release notes.
.github/workflows/draft_release.yml
Outdated
#Validate the tag is formatted correctly YYYY.MM.DD or YYYY.MM.DD-rcX | ||
# where rcX is a whole number greater than zero | ||
if [[ ! $tag =~ ^[0-9]{4}\.[0-9]{2}\.[0-9]{2}(-rc[0-9]+)?$ ]]; then | ||
echo "Invalid tag format. Must be YYYY.MM.DD or YYYY.MM.DD-rcX" | ||
exit 1 | ||
fi |
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.
What's the YYYY.MM.DD-rcX
for? It's new.
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.
Need to support cherry pick releases. 2024.09.05-23 for example. Good catch though, I accidentally included rc in the regex.. will remove.
You generate the changelog when publishing the release. It appends the commits after any other content already on the release doc. It won't override anything. Because releases are essentially immutable, you never change the tag on a release but create a new one, you never should need to update the changelog. Does that clarify? |
There are 2 kinds of cherry-picks though:
|
👍 |
@diox both of your proposed diffs are achievable. I don't think it will be possible to get this granular with the automation, but that is probably okay. If you are making a cherry pick release I think it is up to you to verify your target and compare tags are correct, and you can update or remove the link as you wish. I have included info like this in the PR updating our push duty docs. Do you think this is sufficient? Or would you suggest removing the link? Or trying to make the automation support more complex linking? |
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.
r+ if @diox concurs
Fixes: https://mozilla-hub.atlassian.net/browse/AMOENG-616
Description
Adds a workflow to trigger the creation of a draft release. This allows us to have a standardized template for the information in our releases.
Context
Previously we created release documentation manually and updated with commit diffs of tags and info related to a given release. Now that we can release via github releases we should automate creation of release docs in the same way.
Testing
The criteria for the release is:
Checklist
#ISSUENUM
at the top of your PR to an existing open issue in the mozilla/addons repository.