Skip to content

fix(ci): make DockerHub publish conditional so the dev image builds on forks#690

Merged
javi11 merged 1 commit into
javi11:mainfrom
yoshitaka420:fix/dockerhub-publish-conditional
Jun 15, 2026
Merged

fix(ci): make DockerHub publish conditional so the dev image builds on forks#690
javi11 merged 1 commit into
javi11:mainfrom
yoshitaka420:fix/dockerhub-publish-conditional

Conversation

@yoshitaka420

Copy link
Copy Markdown
Contributor

The dev-image workflow logged in and pushed to DockerHub (laris11/altmount) unconditionally, which fails on forks that lack the DOCKERHUB_USERNAME/DOCKERHUB_TOKEN secrets ("Username and password required").

This gates every DockerHub step on those secrets being present:

  • Forks (no secrets): publish to GHCR only — the dev image builds successfully.
  • Upstream (secrets set): publish to both GHCR and DockerHub, unchanged.

Changes

  • .github/workflows/dev-image.yml: conditionally run the DockerHub login and push steps based on secret availability.

…n forks

The dev-image workflow logged in and pushed to DockerHub (laris11/altmount)
unconditionally, which fails on forks lacking DOCKERHUB_USERNAME/
DOCKERHUB_TOKEN secrets ("Username and password required"). Gate every
DockerHub step on the secrets being present: forks publish to GHCR only,
upstream still publishes to both.
@javi11 javi11 merged commit 7f1d9b7 into javi11:main Jun 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.

2 participants