Skip to content

fix(workflows): stop gitsemver install from building double-v prefix - #244

Merged
piontec merged 2 commits into
mainfrom
fix-gitsemver-double-v-prefix
Jul 16, 2026
Merged

fix(workflows): stop gitsemver install from building double-v prefix#244
piontec merged 2 commits into
mainfrom
fix-gitsemver-double-v-prefix

Conversation

@piontec

@piontec piontec commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Renovate fills the install-binary-action version: field with the v-prefixed release tag (e.g. v2.0.1), but the gitsemver download_url template hardcoded another v (v${version}). The two combined into .../download/vv2.0.1/gitsemver-vv2.0.1-linux-amd64.tar.gz, which 404s and breaks the Create Release workflow for every consumer.

Make the template consume the v from ${version} (matching the neighbouring architect install block) and keep the value v-prefixed, so it resolves correctly and survives future Renovate bumps. This supersedes the temporary fix in eaec78f, which set the value bare and would re-break on the next Renovate bump.

Refs giantswarm/giantswarm#37171

Checklist

  • I have updated the CHANGELOG.md with a description of the change

…d URL

Renovate fills the install-binary-action `version:` field with the
v-prefixed release tag (e.g. `v2.0.1`), but the gitsemver `download_url`
template hardcoded another `v` (`v${version}`). The two combined into
`.../download/vv2.0.1/gitsemver-vv2.0.1-linux-amd64.tar.gz`, which 404s
and breaks the Create Release workflow for every consumer.

Make the template consume the `v` from `${version}` (matching the
neighbouring architect install block) and keep the value `v`-prefixed, so
it resolves correctly and survives future Renovate bumps. This supersedes
the temporary fix in eaec78f, which set the value bare and would re-break
on the next Renovate bump.

Refs giantswarm/giantswarm#37171

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Łukasz Piątkowski <piontec@gmail.com>
@piontec
piontec requested a review from a team as a code owner July 15, 2026 08:21
Comment thread CHANGELOG.md Outdated

@stone-z stone-z left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggest shortening claude's changelog, but otherwise 👍

Co-authored-by: Zach Stone <zach@giantswarm.io>
@piontec
piontec merged commit fc69512 into main Jul 16, 2026
5 checks passed
@piontec
piontec deleted the fix-gitsemver-double-v-prefix branch July 16, 2026 07:48
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.

2 participants