Skip to content

Check Docker build on pull requests from forks - #292

Open
zstadler wants to merge 1 commit into
protomaps:mainfrom
zstadler:docker-build-action-in-PRs
Open

zstadler wants to merge 1 commit into
protomaps:mainfrom
zstadler:docker-build-action-in-PRs

Conversation

@zstadler

Copy link
Copy Markdown
Contributor

Docker Hub login on pull requests from forks always fail. As a result, pull requests from forks are never tested for a funtional docker build prior to their merge.

However, in such cases, this login is not needed since the image is never pushed.

Docker Hub login on pull requests from forks always fail. As a result,
pull requests from forks are never tested for a funtional docker build
prior to their merge.

However, in such cases, this login is not needed since the image is never pushed.
@zstadler

Copy link
Copy Markdown
Contributor Author

This PR also serves as an example where the docker build was tested successfully. On the other hand, no attempt was made to upload the image.

See https://github.com/protomaps/go-pmtiles/actions/runs/40719538810/job/122926343293?pr=292

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