Skip to content

fix: trigger docker and pypi builds on tag push instead of release event#49

Merged
ddulic merged 3 commits intomainfrom
003-github-releases
Mar 16, 2026
Merged

fix: trigger docker and pypi builds on tag push instead of release event#49
ddulic merged 3 commits intomainfrom
003-github-releases

Conversation

@ddulic
Copy link
Copy Markdown
Owner

@ddulic ddulic commented Mar 16, 2026

Problem

docker.yaml and publish.yaml were triggering on release: published, but GitHub suppresses workflow events fired by GITHUB_TOKEN. Since release-please uses GITHUB_TOKEN to create the release, the release: published event never reaches other workflows — so the Docker image and PyPI package were never built on release.

Fix

Switch both workflows to trigger on push: tags: v*.*.*. Release-please pushes the git tag directly (a real push event), which always triggers downstream workflows regardless of the token used.

Flow after this fix

release-please merges release PR
        ↓
pushes tag v1.0.0 (real push event)
        ↓
docker.yaml  → builds + pushes ghcr.io image (v1.0.0 / 1.0 / 1 / latest)
publish.yaml → publishes to PyPI

ddulic and others added 3 commits March 16, 2026 12:20
GitHub Actions does not fire workflow events for actions taken by
GITHUB_TOKEN. Since release-please uses GITHUB_TOKEN to create the
release, the release:published event is suppressed and docker/pypi
workflows never run. Switch both to push:tags:v*.*.* which is a real
push event that always triggers.
@ddulic ddulic merged commit b0e2972 into main Mar 16, 2026
4 checks passed
@ddulic ddulic deleted the 003-github-releases branch March 16, 2026 12:25
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ddulic ddulic mentioned this pull request Mar 16, 2026
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