Merge pull request #82 from truenas/ci-fix

ci: don't run apps test if no changed apps detected
truenas · Jul 18, 2024 · 60516c0 · 60516c0
2 parents d8b5ab0 + 7073002
commit 60516c0
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/.github/workflows/app-test-suite.yaml b/.github/workflows/app-test-suite.yaml
@@ -9,6 +9,7 @@ jobs:
     runs-on: ubuntu-latest
     outputs:
       changed-apps: ${{ steps.changed-apps.outputs.changed-apps }}
+      change-count: ${{ steps.changed-apps.outputs.change-count }}
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -31,11 +32,13 @@ jobs:
         run: |
           out=$(python3 .github/scripts/changed_apps.py)
           echo "changed-apps=${out}" >> $GITHUB_OUTPUT
+          echo "change-count=$(echo "${out}" | jq -r '.include | length')" >> $GITHUB_OUTPUT
 
   run-apps:
     name: Run Docker Compose Render/Install
     needs: changed-files
     runs-on: ubuntu-latest
+    if: needs.changed-files.outputs.change-count > 0
     strategy:
       matrix: ${{ fromJson(needs.changed-files.outputs.changed-apps) }}
       fail-fast: false