Merge pull request #30 from elezar/trigger-on-push

Only use merge event for closed PR
NVIDIA · Nov 21, 2023 · 9323da2 · 9323da2
2 parents 720ca15 + 4058fe3
commit 9323da2
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/.github/workflows/build-images.yaml b/.github/workflows/build-images.yaml
@@ -20,7 +20,6 @@ on:
     types:
       - opened
       - synchronize
-      - closed
     branches:
       - main
       - release-*
@@ -31,10 +30,9 @@ on:
 
 jobs:
   build-image:
-    # We trigger builds on MERGED pull requests or on non-closed PRs that are not created against a fork.
+    # We trigger image builds on push events (MERGED pull requests) or on non-closed PRs that are not created against a fork.
     if: |
-      (github.event_name == 'pull_request' && github.event.action == 'closed' && github.event.pull_request.merged == true) ||
-      (github.event_name == 'pull_request' && github.event.action != 'closed' && github.event.pull_request.head.repo.full_name == github.repository) ||
+      (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository) ||
       (github.event_name == 'push')
     runs-on: ubuntu-latest
     strategy: