Skip to content

Fix PR-safe Docker publish workflow#2

Open
bishoyh wants to merge 1 commit into
mainfrom
fix/maintenance-2026-05-28
Open

Fix PR-safe Docker publish workflow#2
bishoyh wants to merge 1 commit into
mainfrom
fix/maintenance-2026-05-28

Conversation

@bishoyh
Copy link
Copy Markdown
Member

@bishoyh bishoyh commented May 28, 2026

Summary

  • split Docker CI into PR-safe test and push-to-main publish jobs
  • prevent Docker Hub login and image pushes during pull_request validation
  • publish docker.io/picotainers/bowtie2atest only after test passes on main

Validation

  • Parsed .github/workflows/build-publish.yml with Python yaml.safe_load
  • Manually confirmed PR test job has no Docker Hub login and no push
  • Manually confirmed publish needs test and is gated to push on refs/heads/main
  • Local runtime verification pending CI because Docker daemon is unavailable in this environment

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