Merge pull request #42 from ylavoie/push-on-merge-only2

Push onto registry if pull request is merged into master
ledgersmb · Oct 30, 2023 · f2eb3df · f2eb3df
2 parents 2453a37 + 3b3d9e3
commit f2eb3df
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/docker-image.yml b/.github/workflows/docker-image.yml
@@ -26,7 +26,7 @@ jobs:
     - name: Build and push
       uses: docker/build-push-action@v5
       with:
-        push: ${{ github.event.pull_request.merged == true }}
+        push: ${{ github.event_name == 'pull_request' && github.event.action == 'closed' }}
         tags: ghcr.io/${{github.repository_owner}}/ledgersmb_circleci-perl:${{ matrix.version }}
         build-args: |
           perl=${{ matrix.version }}
@@ -52,7 +52,7 @@ jobs:
     - name: Build and push
       uses: docker/build-push-action@v5
       with:
-        push: ${{ github.event.pull_request.merged == true }}
+        push: ${{ github.event_name == 'pull_request' && github.event.action == 'closed' }}
         tags: ghcr.io/${{github.repository_owner}}/ledgersmb_circleci-postgres:${{ matrix.version }}
         build-args: |
           version=${{ matrix.version }}