-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
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
stasadev
left a comment
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.)
|
We shall have tag+release for all upcoming releases (the older tags stasadev found are from the previous repository and we have not added releases for them). But I think @stasadev's change makes sense because AFAIK for Github action versioning, only the tags are important. I'll resolve the conflict with the previous commit, then we can commit and try for our next release. |
Co-authored-by: Stanislav Zhuk <stasadev@gmail.com>
|
Thanks! |
|
There was a problem now when I tagged and pushed the last commit ( Problem was I could also not move the tag manually now! So somebody please move/re-tag the existing This is what happened for me: The mentioned https://github.com/ddev/github-action-setup-ddev/rules?ref=refs%2Ftags%2Fv1 has some ruleset on that tag? I have no rights to adjust there and no idea what it means. I tried to delete https://github.com/ddev/github-action-setup-ddev/releases/tag/v1 does not allow me to click "delete": |
|
Hi @jonaseberle,
Yes, that is our org rule, added as part of: It was originally only for I changed the rule back to repository level in |
|
If that makes sense, I'd just remove tags that don't have releases (except (Or make a release for every existing tag, except |
|
I removed tags that aren't listed here https://github.com/marketplace/actions/setup-ddev-in-github-workflows And re-started the failed workflow, it created a new release |
|
wonderful, thank you @stasadev! |

The Issue
The
v1tag 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
v1release. 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.