-
-
Notifications
You must be signed in to change notification settings - Fork 12
Update branch tags only when releases are made #38
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
base: main
Are you sure you want to change the base?
Conversation
Needs quick rebase and review by @jonaseberle or @stasadev |
Also use https://github.com/tj-actions/release-tagger instead of our own custom workflow code.
6338e7b
to
8ec8f19
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.
I don't think it worked as expected.
I created a fork, merged this branch into main, created a release. It failed, because it goes through each tag, checks for release for each tag to retrieve description.
(This repo has 6 releases and 22 tags - so adding tj-actions/release-tagger
will require to have releases for all tags.)
After that fail, I removed all tags in my fork except v1.10.1
and v1
, created a release for v1.10.1
(because forks don't copy any releases from parent repos), and tried to create a new release v1.10.2
.
It worked, this action successfully added a new release for v1
tag, updated its description and marked it as the latest one.
But, I didn't see a moved tag for v1
, it remained on the same git commit as v1.10.1
.
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.
The suggested change worked for me.
I used the same technique as in the action example.
(And we still need to create releases for all tags.)
on: | ||
release: | ||
types: [published] |
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.
on: | |
release: | |
types: [published] | |
on: | |
push: | |
tags: | |
- v* |
The Issue
The
v1
tag that we have that is always updated should only really be updated when a new release is made, and not on every push to main.How This PR Solves The Issue
Updates the workflow to use https://github.com/tj-actions/release-tagger to update the
v1
release. This also reduces our custom code.Manual Testing Instructions
None
Automated Testing Overview
None
Related Issue Link(s)
Release/Deployment Notes
None, this is an internal workflow change only.