Merge pull request #171 from c4dt/test_docker_build

Only pushing if merge on main
c4dt · Jul 3, 2024 · 572e945 · 572e945
2 parents f95d249 + 0cfc407
commit 572e945
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/.github/workflows/build-docker.yml b/.github/workflows/build-docker.yml
@@ -11,9 +11,9 @@ jobs:
   build-docker:
     name: Build D-Voting Docker images
     runs-on: ubuntu-22.04
-    if: '! github.event.pull_request.draft'
     env:
-      DockerTag: ${{ (github.ref == 'refs/heads/main') && 'latest' || github.head_ref }}
+      DockerTag: latest
+      push: ${{ (github.ref == 'refs/heads/main') && 'true' || 'false' }}
 
     steps:
       - name: Checkout
@@ -30,6 +30,7 @@ jobs:
         id: buildx
         uses: docker/setup-buildx-action@v1
       - name: Login to GHCR
+        if: ${{ env.push == 'true' }}
         uses: docker/login-action@v1
         with:
           registry: ghcr.io
@@ -46,21 +47,21 @@ jobs:
             REACT_APP_VERSION
             REACT_APP_BUILD
             REACT_APP_BUILD_TIME
-          push: true
+          push: ${{ env.push }}
           tags: ghcr.io/c4dt/d-voting-frontend:${{ env.DockerTag }}
       - name: Build Backend
         uses: docker/build-push-action@v2
         with:
           context: .
           file: Dockerfiles/Dockerfile.backend
           platforms: linux/amd64
-          push: true
+          push: ${{ env.push }}
           tags: ghcr.io/c4dt/d-voting-backend:${{ env.DockerTag }}
       - name: Build D-Voting
         uses: docker/build-push-action@v2
         with:
           context: .
           file: Dockerfiles/Dockerfile.dela
           platforms: linux/amd64
-          push: true
+          push: ${{ env.push }}
           tags: ghcr.io/c4dt/d-voting-dela:${{ env.DockerTag }}