Skip to content

Dedupe release triggers to prevent double-publish#21

Merged
CallMarcus merged 1 commit into
mainfrom
claude/issues-status-review-ll7hil
Jun 24, 2026
Merged

Dedupe release triggers to prevent double-publish#21
CallMarcus merged 1 commit into
mainfrom
claude/issues-status-review-ll7hil

Conversation

@CallMarcus

Copy link
Copy Markdown
Owner

Summary

The v1.0.0 release succeeded and is live on PyPI, but the release workflow produced one failed run: publishing a GitHub Release that creates a new tag fires both release: published and push: tags, so the publish job ran twice — the second hit "file already exists" on PyPI.

This dedupes the triggers so future releases produce a single clean run.

Changes (.github/workflows/release.yml)

  • Publish on tag push only. Dropped the release: published trigger. A GitHub Release that creates a new tag still emits the push tag event, so the Release-UI flow keeps working — as a single run. CLI git tag … && git push is unchanged.
  • skip-existing: true on both publish steps as a safety net against duplicate uploads (e.g. manual re-runs of a run).

Behavior after merge

Flow Result
git tag v1.2.3 && git push origin v1.2.3 1 run → publishes
GitHub Release UI (create new tag on publish) 1 run → publishes
Manual "Run workflow" publishes to TestPyPI

No change needed to the pending publisher (still release.yml / env pypi).

🤖 Generated with Claude Code


Generated by Claude Code

Publishing a GitHub Release that creates a new tag fires both the
`release: published` and `push: tags` events, so v1.0.0 ran the publish job
twice and the second run failed with "file already exists".

Trigger publishing on tag push only — a Release that creates a new tag still
fires this event, so the Release UI flow keeps working as a single run. Also
add `skip-existing: true` to both publish steps as a safety net against
duplicate uploads (e.g. manual re-runs).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XYvSCw1sjoG2AHbLPCsX7S
@CallMarcus CallMarcus mentioned this pull request Jun 22, 2026
41 tasks
@CallMarcus CallMarcus merged commit 7b7b107 into main Jun 24, 2026
17 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.

2 participants