Prevent duplicate actions from running. (#252)

* Use skip-duplicate-actions to avoid running same actions multiple times concurrently. * Fix test workflow to only run on Ubuntu (latest).
AjayKMehta · Feb 24, 2025 · ea1fac8 · ea1fac8
1 parent f513ab7
commit ea1fac8
Show file tree

Hide file tree

Showing 4 changed files with 69 additions and 6 deletions.
diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
@@ -22,7 +22,24 @@ permissions:
   contents: write
 
 jobs:
+
+  pre_job:
+    # continue-on-error: true # Uncomment once integration is finished
+    runs-on: ubuntu-latest
+    # Map a step output to a job output
+    outputs:
+      should_skip: ${{ steps.skip_check.outputs.should_skip }}
+    steps:
+      - id: skip_check
+        uses: fkirc/skip-duplicate-actions@v5
+        with:
+          cancel_others: 'true'
+          skip_after_successful_duplicate: 'true'
+          concurrent_skipping: same_content_newer
+
   analyze:
+    needs: pre_job
+    if: needs.pre_job.outputs.should_skip != 'true'
     name: Analyze (${{ matrix.language }})
     runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-latest' }}
     timeout-minutes: ${{ (matrix.language == 'swift' && 120) || 360 }}

diff --git a/.github/workflows/dependency-review.yml b/.github/workflows/dependency-review.yml
@@ -12,8 +12,23 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
+  pre_job:
+    # continue-on-error: true # Uncomment once integration is finished
+    runs-on: ubuntu-latest
+    # Map a step output to a job output
+    outputs:
+      should_skip: ${{ steps.skip_check.outputs.should_skip }}
+    steps:
+      - id: skip_check
+        uses: fkirc/skip-duplicate-actions@v5
+        with:
+          cancel_others: 'true'
+          skip_after_successful_duplicate: 'true'
+          concurrent_skipping: same_content_newer
+
   dependency-review:
-    if: github.actor != 'dependabot[bot]'
+    needs: pre_job
+    if: github.actor != 'dependabot[bot]' && needs.pre_job.outputs.should_skip != 'true'
     runs-on: ubuntu-latest
     steps:
       - name: 'Checkout Repository'

diff --git a/.github/workflows/semgrep.yml b/.github/workflows/semgrep.yml
@@ -15,9 +15,27 @@ on:
         description: "The reason for running the workflow"
         required: true
         default: "Manual run"
+
 name: Semgrep
+
 jobs:
+  pre_job:
+    # continue-on-error: true # Uncomment once integration is finished
+    runs-on: ubuntu-latest
+    # Map a step output to a job output
+    outputs:
+      should_skip: ${{ steps.skip_check.outputs.should_skip }}
+    steps:
+      - id: skip_check
+        uses: fkirc/skip-duplicate-actions@v5
+        with:
+          cancel_others: 'true'
+          skip_after_successful_duplicate: 'true'
+          concurrent_skipping: same_content_newer
+
   semgrep:
+    needs: pre_job
+    if: needs.pre_job.outputs.should_skip != 'true'
     name: Semgrep/CI
     runs-on: ubuntu-latest
     env:

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -18,13 +18,26 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
+  pre_job:
+    # continue-on-error: true # Uncomment once integration is finished
+    runs-on: ubuntu-latest
+    # Map a step output to a job output
+    outputs:
+      should_skip: ${{ steps.skip_check.outputs.should_skip }}
+    steps:
+      - id: skip_check
+        uses: fkirc/skip-duplicate-actions@v5
+        with:
+          cancel_others: 'true'
+          skip_after_successful_duplicate: 'true'
+          concurrent_skipping: same_content_newer
+
   test:
-    name: test-${{matrix.os}}
-    runs-on: ${{ matrix.os }}
+    needs: pre_job
+    if: needs.pre_job.outputs.should_skip != 'true'
+    name: test
+    runs-on: ubuntu-latest
     timeout-minutes: 10
-    strategy:
-      matrix:
-        os: [ubuntu-latest]
     env:
       NUGET_PACKAGES: ${{ github.workspace }}/.nuget/packages
       DOTNET_NOLOGO: true