build: more readable syntax for check conditions

nedbat · nedbat · commit 99cb3c773f1f · 2025-12-09T06:39:21.000-05:00
As suggested by Adam Johnson in a comment here: https://nedbatchelder.com/blog/202505/filtering_github_actions_by_changed_files.html
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -67,7 +67,8 @@ jobs:
 
     # Only run coverage if Python files or this workflow changed.
     needs: changed
-    if: ${{ needs.changed.outputs.run_coverage == 'true' }}
+    if:
+      needs.changed.outputs.run_coverage == 'true'
 
     env:
       MATRIX_ID: "${{ matrix.python-version }}.${{ matrix.os }}"
diff --git a/.github/workflows/quality.yml b/.github/workflows/quality.yml
@@ -67,7 +67,10 @@ jobs:
     runs-on: macos-latest
 
     needs: changed
-    if: ${{ needs.changed.outputs.python == 'true' || needs.changed.outputs.actions == 'true' || needs.changed.outputs.workflow == 'true' }}
+    if:
+      needs.changed.outputs.python == 'true'
+      || needs.changed.outputs.actions == 'true'
+      || needs.changed.outputs.workflow == 'true'
 
     steps:
       - name: "Check out the repo"
@@ -95,7 +98,9 @@ jobs:
     runs-on: ubuntu-latest
 
     needs: changed
-    if: ${{ needs.changed.outputs.python == 'true' || needs.changed.outputs.workflow == 'true' }}
+    if:
+      needs.changed.outputs.python == 'true'
+      || needs.changed.outputs.workflow == 'true'
 
     steps:
       - name: "Check out the repo"
@@ -123,7 +128,9 @@ jobs:
     runs-on: ubuntu-latest
 
     needs: changed
-    if: ${{ needs.changed.outputs.docs == 'true' || needs.changed.outputs.workflow == 'true' }}
+    if:
+      needs.changed.outputs.docs == 'true'
+      || needs.changed.outputs.workflow == 'true'
 
     steps:
       - name: "Check out the repo"
@@ -166,7 +173,9 @@ jobs:
       contents: read
 
     needs: changed
-    if: ${{ needs.changed.outputs.actions == 'true' || needs.changed.outputs.workflow == 'true' }}
+    if:
+      needs.changed.outputs.actions == 'true'
+      || needs.changed.outputs.workflow == 'true'
 
     steps:
       - name: Checkout repository
diff --git a/.github/workflows/testsuite.yml b/.github/workflows/testsuite.yml
@@ -65,7 +65,9 @@ jobs:
     # Don't run tests if the branch name includes "-notests".
     # Only run tests if files that affect tests have changed.
     needs: changed
-    if: ${{ needs.changed.outputs.run_tests == 'true' && !contains(github.ref, '-notests') }}
+    if:
+      needs.changed.outputs.run_tests == 'true'
+      && !contains(github.ref, '-notests')
 
     env:
       TOX_GH_MAJOR_MINOR: "${{ matrix.python-version }}"
