Skip to content

Fix release.yml tag job: set git identity for annotated tag#165

Merged
alexkroman merged 1 commit into
mainfrom
claude/release-script-x098mt
Jun 14, 2026
Merged

Fix release.yml tag job: set git identity for annotated tag#165
alexkroman merged 1 commit into
mainfrom
claude/release-script-x098mt

Conversation

@alexkroman

Copy link
Copy Markdown
Collaborator

Problem

The manual workflow_dispatch release path in .github/workflows/release.yml fails immediately in the resolve release tag job:

==> Ready to release v0.1.6 at bfb3355 (main, in sync with origin).
*** Please tell me who you are.
fatal: empty ident name (for <runner@...>) not allowed
##[error]Process completed with exit code 128.

The tag job checks out with persist-credentials: false and never configures a git identity, but cut_release.sh cuts an annotated tag (git tag -a), which requires a committer identity. As a result the bottle and publish jobs are skipped and no tag/release is produced. This path has never succeeded from CI — it only works locally because maintainers have a global git identity.

Observed in run 27485734710.

Fix

Set the github-actions[bot] git identity in the tag job before invoking cut_release.sh, mirroring what the publish job already does before its commit. Workflow-only change; actionlint clean and the full gate passes.

After merge

Dispatch the Release workflow (blank version) to cut v0.1.6 from main.

https://claude.ai/code/session_016ZF4HUbyCpwHH9CG6X6NS3


Generated by Claude Code

The manual workflow_dispatch release path failed at the 'resolve release
tag' step with 'fatal: empty ident name … exit code 128'. The job checks
out with persist-credentials: false and sets no git identity, but
cut_release.sh cuts an annotated tag (git tag -a), which requires a
committer identity. Set the github-actions[bot] identity before calling
the script, matching what the publish job already does.
@alexkroman alexkroman enabled auto-merge June 14, 2026 02:53
@alexkroman alexkroman added this pull request to the merge queue Jun 14, 2026
Merged via the queue into main with commit b9d575e Jun 14, 2026
19 checks passed
@alexkroman alexkroman deleted the claude/release-script-x098mt branch June 14, 2026 02:58
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