bad linting but correct I think, escaping is hard

firedrakeproject · Jan 16, 2025 · 8b82c4d · 8b82c4d
1 parent 07b57d7
commit 8b82c4d
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -45,7 +45,7 @@ jobs:
       OPENBLAS_NUM_THREADS: 1
       COMPLEX: ${{ matrix.complex }}
       RDMAV_FORK_SAFE: 1
-      PYTEST_ARGS: --group {#} --splitting-algorithm least_duration --timeout=1800 --timeout-method=thread -o faulthandler_timeout=1860 --verbose
+      PYTEST_ARGS: 
     outputs:
       scalar-type: ${{ matrix.scalar-type }}
     steps:
@@ -103,14 +103,20 @@ jobs:
           . ../firedrake_venv/bin/activate
           : # use --quote to stop parallel from parsing the pytest arguments
           parallel --line-buffer --tag --quote \
-            pytest "$PYTEST_ARGS" --splits 12 --junit-xml="firedrake1_{#}.xml" -m "parallel[1] or not parallel" tests/firedrake ::: $(seq 12)
+            pytest -v --splits 12 --group {#} --splitting-algorithm least_duration \
+            --timeout=1800 --timeout-method=thread -o faulthandler_timeout=1860 \
+            --junit-xml=firedrake1_{#}.xml \
+            -m "parallel[1] or not parallel" tests/firedrake ::: $(seq 12)
       - name: Run tests (nprocs = 2)
         # Run even if earlier tests failed
         if: ${{ success() || steps.build.conclusion == 'success' }}
         run: |
           . ../firedrake_venv/bin/activate
           parallel --line-buffer --tag --quote \
-            mpiexec -n 2 pytest "$PYTEST_ARGS" --splits 6 --junit-xml="firedrake2_{#}.xml" -m parallel[2] tests/firedrake ::: $(seq 6)
+            mpiexec -n 2 pytest -v --splits 6 --group {#} --splitting-algorithm least_duration \
+            --timeout=1800 --timeout-method=thread -o faulthandler_timeout=1860 \
+            --junit-xml=firedrake2_{#}.xml \
+            -m parallel[2] tests/firedrake ::: $(seq 6)
 
       # - name: Run tests (nprocs = 3)
       #   if: ${{ success() || steps.build.conclusion == 'success' }}