ci: fix release-reusable git identity and add manual dispatch - #25
Merged
Conversation
The first run failed at `git tag -a` with "empty ident name" — the runner has no git user configured. Set the github-actions[bot] identity before tagging. Also add workflow_dispatch (with a bump input) so a release can be cut manually, and factor the bump level out so push and dispatch share one code path.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
The first
release-reusablerun (from #24) failed atgit tag -awithfatal: empty ident name— the runner has no git user configured, so annotated tags can't be created. The version math ran fine (Releasing v1.0.0 (previous v0.0.0)) and it failed before any tag or push, so nothing partial landed.This:
github-actions[bot]git identity before tagging;workflow_dispatchwith abump(patch/minor/major) input so a release can be cut on demand — needed right now because this fix commit only touchesrelease-reusable.yml, which isn't areusable-*trigger path, so it won't auto-fire;After merge I'll dispatch it with
bump: majorto cut the inauguralv1.0.0(+v1), then re-pin the forks onto that tag.Type of change
fix/feat/perf— consumer-facingdeps— dependency update (add thesecuritylabel if it fixes a CVE)chore/ci/build/docs/test/refactor— maintenanceChecklist
commitlintpasses)npm testpasses on Node 20 / 22 / 24 — workflow-only change, no package surfacenode:testover new runners)Generated by Claude Code