Skip to content

fix: run Docker build in the same run as release-please#5

Merged
bin101 merged 1 commit into
mainfrom
develop
Jul 10, 2026
Merged

fix: run Docker build in the same run as release-please#5
bin101 merged 1 commit into
mainfrom
develop

Conversation

@bin101

@bin101 bin101 commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes the gap you hit: v0.2.0 got tagged + released but no GHCR image was built. Root cause: publish.yml's on: push: tags: trigger can never fire for a tag that release-please-action itself creates, because it uses the default GITHUB_TOKEN -- and GitHub Actions deliberately blocks refs created with that token from triggering further workflow runs (recursion guard).

Fix: the Docker build now runs as a build job inside release-please.yml itself (gated on the release-please job's release_created output), in the same run that already has permission to act -- no second trigger involved. publish.yml is now a workflow_dispatch-only fallback for manually re-publishing an existing tag.

Test plan

  • pytest -q — 49 passed
  • YAML for all three workflow files parses cleanly
  • After merge: confirm the next release-please release actually produces a GHCR image in the same run
  • Backfill: manually dispatch publish.yml with tag: v0.2.0 to publish the image that's currently missing for the existing release

🤖 Generated with Claude Code

The v0.2.0 release cut a tag and GitHub Release with no GHCR image to show
for it: publish.yml's `on: push: tags:` trigger never fired, because
GitHub Actions doesn't let a ref created via the default GITHUB_TOKEN
(which is how release-please-action creates its tag) trigger another
workflow run -- a deliberate recursion guard, not a bug.

Moves the Docker build into release-please.yml itself, as a `build` job
gated on the release-please job's release_created output, so it runs in
the same run that already has permission to act (no separate trigger
needed). publish.yml becomes a workflow_dispatch-only fallback for
re-publishing an existing tag on demand.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@bin101 bin101 merged commit 767aa20 into main Jul 10, 2026
2 checks passed
@bin101 bin101 deleted the develop branch July 10, 2026 07:23
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