Skip to content

Fix tag pushing and GitHub release creation#4

Merged
theoephraim merged 3 commits into
mainfrom
fix/tag-push-and-release
Apr 15, 2026
Merged

Fix tag pushing and GitHub release creation#4
theoephraim merged 3 commits into
mainfrom
fix/tag-push-and-release

Conversation

@theoephraim

Copy link
Copy Markdown
Member

Summary

  • git push --follow-tags only pushes annotated tags reachable from pushed commits — bumpy creates lightweight tags and may have no new commits to push, so tags were silently not being pushed
  • gh release create then failed with: tag exists locally but has not been pushed
  • Switch to git push + git push --tags to ensure lightweight tags are pushed
  • Pass --target <SHA> to gh release create as a fallback in case tags haven't propagated

Test plan

  • Merge and trigger a publish — verify tags appear on remote and GitHub releases are created
  • Verify bumpy publish --dry-run still logs expected tag/release info

🐸 Generated with Claude Code

@github-actions

Copy link
Copy Markdown

🐸 Bumpy Release Plan

1 changeset(s) → 1 package(s) to release

🟢 Patch

Package Change
@varlock/bumpy 0.0.1 → 0.0.2

This comment is maintained by bumpy.

@theoephraim theoephraim force-pushed the fix/tag-push-and-release branch from 66198bd to e49118b Compare April 15, 2026 04:56
`git push --follow-tags` only pushes annotated tags reachable from pushed
commits. Since bumpy creates lightweight tags and may have no new commits,
tags were silently not being pushed, causing `gh release create` to fail
with "tag exists locally but has not been pushed".

- Switch to `git push` + `git push --tags` to ensure lightweight tags are pushed
- Pass `--target <SHA>` to `gh release create` as a fallback

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@theoephraim theoephraim force-pushed the fix/tag-push-and-release branch from e49118b to 178f8c3 Compare April 15, 2026 05:01
theoephraim and others added 2 commits April 14, 2026 22:13
Append -n suffix to aggregate release tags when multiple releases happen
on the same day. Extract resolveAggregateTagAndTitle as a pure function.
Add test coverage for git helpers and github release logic.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The test failed in CI because `git push` requires a tracking branch.
Add an initial `git push -u origin HEAD` to set up tracking.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@theoephraim theoephraim merged commit d219d71 into main Apr 15, 2026
2 checks passed
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