Merge pull request #2006 from pllim/fix-cancel-ci

TST: Fix cancel workflow for PRs

Author: keflavich

.github/workflows/cancel_workflows.yml

@@ -0,0 +1,17 @@
+name: Cancel duplicate workflows
+
+on:
+  workflow_run:
+    workflows: ["CI", "CodeQL"]
+    types:
+      - requested
+
+# Note: This has to be in workflow_run so it works for PRs from forks.
+jobs:
+  cancel:
+    runs-on: ubuntu-latest
+    steps:
+    - name: Cancel previous runs
+      uses: styfle/cancel-workflow-action@3d86a7cc43670094ac248017207be0295edbc31d  # 0.8.0
+      with:
+        workflow_id: ${{ github.event.workflow.id }}

.github/workflows/ci_tests.yml

@@ -14,22 +14,9 @@ on:
     - cron: '0 5 * * 1'
 
 jobs:
-  cancel_ci:
-    name: Mandatory checks before CI
-    runs-on: ubuntu-latest
-    steps:
-    # TODO: Fix this for PRs
-    - name: Cancel Previous Runs
-      uses: styfle/cancel-workflow-action@ce177499ccf9fd2aded3b0426c97e5434c2e8a73
-      with:
-        access_token: ${{ secrets.GITHUB_TOKEN }}
-
-  # The rest only run if above are done (unless cron only)
-
   tests:
     name: ${{ matrix.name }}
     runs-on: ${{ matrix.os }}
-    needs: cancel_ci
     strategy:
       fail-fast: true
       matrix:
@@ -113,7 +100,6 @@ jobs:
   egg_info:
     name: egg_info with Python 3.7
     runs-on: ubuntu-latest
-    needs: cancel_ci
     steps:
     - name: Checkout code
       uses: actions/checkout@v2

.github/workflows/codeql-analysis.yml

@@ -19,22 +19,9 @@ on:
     branches: [ master ]
 
 jobs:
-  cancel_ci:
-    name: Mandatory checks before CI
-    runs-on: ubuntu-latest
-    steps:
-    # TODO: Fix this for PRs
-    - name: Cancel Previous Runs
-      uses: styfle/cancel-workflow-action@ce177499ccf9fd2aded3b0426c97e5434c2e8a73
-      with:
-        access_token: ${{ secrets.GITHUB_TOKEN }}
-
-  # The rest only run if above are done
-
   analyze:
     name: Analyze
     runs-on: ubuntu-latest
-    needs: cancel_ci
 
     strategy:
       fail-fast: false