Skip to content

Build releases before tagging so tags are immutable#22

Merged
roborourke merged 1 commit into
mainfrom
claude/build-release-process-29djgt
Jun 12, 2026
Merged

Build releases before tagging so tags are immutable#22
roborourke merged 1 commit into
mainfrom
claude/build-release-process-29djgt

Conversation

@roborourke

@roborourke roborourke commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

The release workflow previously created the tag (via the GitHub release UI)
and then force-pushed a rewritten tag to stamp the version and add built
assets. Packagist requires tags to be immutable and rejected the update.

Rework the Release workflow to build and stamp the version BEFORE the tag
exists, then create the vX.Y.Z tag once at the built+versioned commit and
push it without force. Generate the ZIP with git archive so .gitattributes
is the single source of truth for release contents.

Supercedes #21

The release workflow previously created the tag (via the GitHub release UI)
and then force-pushed a rewritten tag to stamp the version and add built
assets. Packagist requires tags to be immutable and rejected the update.

Rework the Release workflow to build and stamp the version BEFORE the tag
exists, then create the vX.Y.Z tag once at the built+versioned commit and
push it without force. Generate the ZIP with git archive so .gitattributes
is the single source of truth for release contents.
@roborourke roborourke requested a review from mattheu June 12, 2026 14:04
@roborourke

Copy link
Copy Markdown
Collaborator Author

Go fable :)

@mattheu mattheu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Think this makes sense.

@roborourke roborourke merged commit eaca7fd into main Jun 12, 2026
1 check failed
@roborourke roborourke deleted the claude/build-release-process-29djgt branch June 12, 2026 15:38
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.

3 participants