workaround for skipping matrix job not reporting subjobs

See actions/runner#952 and https://github.com/orgs/community/discussions/9141
authzed · Jul 17, 2023 · 6545798 · 6545798
1 parent 0f43ff7
commit 6545798
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/.github/workflows/build-test.yaml b/.github/workflows/build-test.yaml
@@ -101,36 +101,44 @@ jobs:
     name: "Datastore Tests"
     runs-on: "ubuntu-latest"
     needs: "paths-filter"
-    if: |
-      needs.paths-filter.outputs.codechange == 'true'
     strategy:
       fail-fast: false
       matrix:
         datastore: ["crdb", "mysql", "postgres", "spanner"]
     steps:
       - uses: "actions/checkout@v3"
+        if: |
+          needs.paths-filter.outputs.codechange == 'true'
       - uses: "authzed/actions/setup-go@main"
+        if: |
+          needs.paths-filter.outputs.codechange == 'true'
         with:
           go-version: "${{ env.GO_VERSION }}"
       - name: "Integration tests"
+        if: |
+          needs.paths-filter.outputs.codechange == 'true'
         run: "go run mage.go testds:${{ matrix.datastore }}"
 
   datastoreconsistency:
     name: "Datastore Consistency Tests"
     runs-on: "ubuntu-latest"
     needs: "paths-filter"
-    if: |
-      needs.paths-filter.outputs.codechange == 'true'
     strategy:
       fail-fast: false
       matrix:
         datastore: ["crdb", "mysql", "postgres", "spanner"]
     steps:
       - uses: "actions/checkout@v3"
+        if: |
+          needs.paths-filter.outputs.codechange == 'true'
       - uses: "authzed/actions/setup-go@main"
+        if: |
+          needs.paths-filter.outputs.codechange == 'true'
         with:
           go-version: "${{ env.GO_VERSION }}"
       - name: "Integration tests"
+        if: |
+          needs.paths-filter.outputs.codechange == 'true'
         run: "go run mage.go testcons:${{ matrix.datastore }}"
 
   e2e: