Skip to content

fix(ci): create release tags on the pushed commit instead of default branch - #4983

Merged
Siumauricio merged 1 commit into
canaryfrom
fix/release-tag-target-commitish
Aug 6, 2026
Merged

fix(ci): create release tags on the pushed commit instead of default branch#4983
Siumauricio merged 1 commit into
canaryfrom
fix/release-tag-target-commitish

Conversation

@Siumauricio

@Siumauricio Siumauricio commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

softprops/action-gh-release creates the version tag without target_commitish, so the GitHub API places it on the default branch (canary) instead of the commit on main that was actually built. This is why the v0.29.14 tag pointed to a canary commit and its auto-generated notes listed unreleased features. target_commitish: ${{ github.sha }} pins the tag to the exact commit the workflow ran on.

Greptile Summary

Pins newly created release tags to the exact workflow commit rather than the repository’s default branch.

  • Adds target_commitish: ${{ github.sha }} to the GitHub release action.
  • Keeps release tags aligned with the commit used by the build and release jobs.

Confidence Score: 5/5

The PR appears safe to merge with no blocking or non-blocking issues identified.

The explicitly targeted commit is the same commit used for checkout, version extraction, and image builds, preserving release provenance across supported triggers.

Reviews (1): Last reviewed commit: "fix(ci): create release tags on the push..." | Re-trigger Greptile

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 6, 2026
@Siumauricio
Siumauricio merged commit 1f04ebf into canary Aug 6, 2026
6 checks passed
@Siumauricio
Siumauricio deleted the fix/release-tag-target-commitish branch August 6, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant