Skip to content

Commit 38df383

Browse files
sbryngelsonclaude
andcommitted
ci: fix --precision flag and remove Python 3.14 step in github job
--${{ matrix.precision }} expands to bare '--' when precision is empty, which is parsed as an argument separator. Restore the PRECISION env var approach so empty precision is correctly omitted. Remove the manual 'Set up Python 3.14' step; the lint-gate job already pins Python 3.12 via setup-python and the github runners use the system default for the build/test steps. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 0819b0e commit 38df383

1 file changed

Lines changed: 2 additions & 6 deletions

File tree

.github/workflows/test.yml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -137,16 +137,12 @@ jobs:
137137
printenv | sort > /tmp/env_after
138138
diff /tmp/env_before /tmp/env_after | grep '^>' | sed 's/^> //' >> $GITHUB_ENV
139139
140-
- name: Set up Python 3.14
141-
uses: actions/setup-python@v5
142-
with:
143-
python-version: '3.14'
144-
145140
- name: Build
146141
run: |
147-
/bin/bash mfc.sh test -v --dry-run -j $(nproc) --${{ matrix.debug }} --${{ matrix.mpi }} --${{ matrix.precision }} $TEST_ALL
142+
/bin/bash mfc.sh test -v --dry-run -j $(nproc) --${{ matrix.debug }} --${{ matrix.mpi }} $PRECISION $TEST_ALL
148143
env:
149144
TEST_ALL: ${{ matrix.mpi == 'mpi' && '--test-all' || '' }}
145+
PRECISION: ${{ matrix.precision != '' && format('--{0}', matrix.precision) || '' }}
150146

151147
- name: Test
152148
run: |

0 commit comments

Comments
 (0)