ci: enhance if-expressions and matrix settings

.github/workflows/test.yml

@@ -153,16 +153,19 @@ jobs:
       matrix:
         include:
         - runs-on: ubuntu-latest
-          shell: bash
+          diffs: "ctex/build/**/*.diff"
           font-dir: /usr/share/fonts/truetype
+          shell: bash
           tl-pkgs: ""
         - runs-on: macos-14
-          shell: bash
+          diffs: "ctex/build/**/*.diff"
           font-dir: /Library/Fonts
+          shell: bash
           tl-pkgs: ""
         - runs-on: windows-latest
-          shell: C:\msys64\usr\bin\bash.exe -e {0}
+          diffs: "ctex/build/**/*.fc"
           font-dir: /c/Windows/Fonts
+          shell: C:\msys64\usr\bin\bash.exe -e {0}
           # for `unzip` shipped with texlive
           tl-pkgs: wintools.windows
 
@@ -178,9 +181,10 @@ jobs:
     - uses: actions/checkout@v4
 
     - name: Install TeX Live
+      timeout-minutes: 30 # fail fast
       uses: teatimeguest/setup-texlive-action@v3
       with:
-        packages: >-
+        packages: |
           ${{ env.TL_PACKAGES }}
           ${{ env.TL_EXTRA_PACKAGES }}
           ${{ matrix.tl-pkgs }}
@@ -193,22 +197,24 @@ jobs:
         unzip -ojd ${{ matrix.font-dir }} "*OTC.zip" "*.ttc"
 
     - name: Test ctex
+      id: test
       working-directory: ./ctex
       run: |
         l3build check -q -H
         l3build check -c test/config-cmap -q -H
         l3build check -c test/config-contrib -q -H
 
     - name: Get short commit SHA
-      if: failure()
+      # if-expression starting with "!" must be escaped,
+      # since "!" is reserved notation in YAML
+      if: ${{ !cancelled() && steps.test.outcome == 'failure' }}
       run: |
         echo "SHA_SHORT=$(git rev-parse --short=8 HEAD)" >> $GITHUB_ENV
 
     - name: Upload test diffs
-      if: failure()
+      if: ${{ !cancelled() && steps.test.outcome == 'failure' }}
       uses: actions/upload-artifact@v4
       with:
         name: ctex-kit-diff-${{ env.SHA_SHORT }}
         path: |
-          ctex/build/**/*.diff
-          ctex/build/**/*.fc
+          ${{ matrix.diffs }}