Skip to content

Conversation

@jovnc
Copy link
Collaborator

@jovnc jovnc commented Jan 11, 2026

Fixes git-mastery/git-mastery#57

Problem

The arch-publish GitHub Action publishes an incorrect AMD64 release download URL, which breaks installation on Arch (e.g., via yay gitmastery-bin). Based on workflow logs, releases starting from ≥7.1.3 are affected.

image

It is fetching from the following URL:
https://github.com/git-mastery/app/releases/download/main/gitmastery-7.1.4-linux-amd64

Instead, it should be fetching from this URL:
https://github.com/git-mastery/app/releases/download/7.1.4/gitmastery-7.1.4-linux-amd64

Root Cause

arch-publish is triggered via workflow_run. In this context, ${{ github.ref_name }} resolves to the branch name that triggered the upstream workflow (e.g., main) rather than the release tag.

This causes the action to construct download URLs using main instead of the version tag, even though the actual package version is correctly determined (e.g., logs show pkgver="v7.1.4").

Fix

Replace ${{ github.ref_name }} with needs.prepare.outputs.ref_name (the tag value)

@jovnc jovnc requested a review from woojiahao January 11, 2026 03:59
@jovnc jovnc added bug Something isn't working bump:patch Automatically creates a tag for a patch release bump on merge labels Jan 11, 2026
Copy link
Member

@woojiahao woojiahao left a comment

Choose a reason for hiding this comment

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

Great catch, thank you for resolving this

@woojiahao woojiahao added bump:minor Automatically creates a tag for a minor release bump on merge and removed bump:patch Automatically creates a tag for a patch release bump on merge labels Jan 12, 2026
@woojiahao
Copy link
Member

Changing release type to minor since we would want our Arch users to update. We should figure out a good way to reason about versioning and what changes are treated as patches

@woojiahao woojiahao merged commit 923182b into git-mastery:main Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working bump:minor Automatically creates a tag for a minor release bump on merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Arch package is broken due to updated release link

2 participants