Skip to content

Commit

Permalink
Merge pull request #114 from tarmac-project/docker-build
Browse files Browse the repository at this point in the history
chore(ci): fixing the docker build i broke again
  • Loading branch information
madflojo authored May 25, 2024
2 parents 0edee39 + ac3893d commit b0617fb
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:
- main
tags:
- v*
pull_request:
pull_request:

jobs:
build:
Expand All @@ -27,6 +27,13 @@ jobs:
with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_TOKEN }}
- name: Build only
id: docker_build_only
uses: docker/build-push-action@v2
with:
platforms: linux/amd64,linux/arm64
push: false
tags: madflojo/tarmac:${{ github.sha }}
- name: Build and push (main)
id: docker_build
if: github.ref == 'refs/heads/main'
Expand All @@ -41,7 +48,7 @@ jobs:
uses: docker/build-push-action@v2
with:
platforms: linux/amd64,linux/arm64
push: ${{ github.event_name == 'push', 'true', 'false' }}
push: ${{ github.event_name == 'push' && 'true' || 'false' }}
# tag with the git tag v*
tags: |
madflojo/tarmac:${{ github.ref_name }}
Expand Down

0 comments on commit b0617fb

Please sign in to comment.