Skip to content

fix: commit changelog before tagging to fix goreleaser dirty-state error#36

Merged
nusmql merged 1 commit intomainfrom
fix/release-dirty-state
Apr 9, 2026
Merged

fix: commit changelog before tagging to fix goreleaser dirty-state error#36
nusmql merged 1 commit intomainfrom
fix/release-dirty-state

Conversation

@nusmql
Copy link
Copy Markdown
Collaborator

@nusmql nusmql commented Apr 9, 2026

Summary

  • The release workflow generates CHANGELOG.md and RELEASE_NOTES.md but never committed them before creating the git tag
  • Goreleaser's dirty-state check catches the uncommitted files and aborts the release
  • Now commits both files before tagging, so goreleaser runs on a clean working tree
  • Removes the redundant changelog PR step since the changelog is committed directly to main

Test plan

  • Trigger a release via workflow_dispatch and verify goreleaser completes successfully

🤖 Generated with Claude Code

The release workflow generated CHANGELOG.md and RELEASE_NOTES.md but
never committed them before creating the git tag. Goreleaser's dirty-state
check caught the uncommitted files and aborted the release.

Now commits both files before tagging, and removes the redundant
changelog PR step since the changelog is committed directly to main.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nusmql nusmql merged commit 897ef0e into main Apr 9, 2026
12 checks passed
@nusmql nusmql deleted the fix/release-dirty-state branch April 9, 2026 19:07
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.

1 participant