Skip to content

ci: build the image on every push, push it only for main, tags and pr… - #260

Open
sandragjacinto wants to merge 1 commit into
mainfrom
ci/verify-image-build
Open

sandragjacinto wants to merge 1 commit into
mainfrom
ci/verify-image-build

Conversation

@sandragjacinto

Copy link
Copy Markdown
Collaborator

…eview PRs

A broken Dockerfile used to show up only after the merge, since #258 builds the image only for deploy targets. build-docker now runs on every push (not Dependabot's, which gets no secrets), with push set from deploy-target: main, tags, preview-labelled PRs and manual runs push the image and get a deploy tag; any other branch only builds it, reading the registry cache without writing it.

Needs the push input of CyberCRI/github-workflows build-push.yaml.

Description

Why?

How?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My code is tested.
  • I have updated the documentation accordingly.

…eview PRs

A broken Dockerfile used to show up only after the merge, since #258 builds
the image only for deploy targets. build-docker now runs on every push (not
Dependabot's, which gets no secrets), with push set from deploy-target:
main, tags, preview-labelled PRs and manual runs push the image and get a
deploy tag; any other branch only builds it, reading the registry cache
without writing it.

Needs the push input of CyberCRI/github-workflows build-push.yaml.
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