Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

davereid
Copy link
Collaborator

@davereid davereid commented May 8, 2025

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.

@davereid davereid requested a review from jonaseberle as a code owner May 8, 2025 15:29
@rfay
Copy link
Member

rfay commented Sep 8, 2025

Needs quick rebase and review by @jonaseberle or @stasadev

@stasadev stasadev force-pushed the update-release-workflow branch from 6338e7b to 8ec8f19 Compare September 8, 2025 17:01
Copy link
Member

@stasadev stasadev left a 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.

Copy link
Member

@stasadev stasadev left a 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.)

Comment on lines +6 to +8
on:
release:
types: [published]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
on:
release:
types: [published]
on:
push:
tags:
- v*

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.

3 participants