Skip to content

Commit eec491a

Browse files
committed
Fix condition for Python coverage
1 parent 90e08e7 commit eec491a

File tree

1 file changed

+12
-27
lines changed

1 file changed

+12
-27
lines changed

.github/workflows/ci.yml

Lines changed: 12 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,15 @@ jobs:
172172
runs-on: ${{ matrix.runs-on }}
173173
setup-z3: true
174174

175+
python-coverage:
176+
name: 🐍 Coverage
177+
needs: [change-detection, python-tests]
178+
if: fromJSON(needs.change-detection.outputs.run-python-tests)
179+
uses: munich-quantum-toolkit/workflows/.github/workflows/[email protected]
180+
permissions:
181+
contents: read
182+
id-token: write
183+
175184
# run extensive Python tests on PRs labeled with the `extensive-python-ci` label
176185
python-tests-extensive:
177186
name: 🐍 Test (Extensive)
@@ -180,32 +189,12 @@ jobs:
180189
strategy:
181190
fail-fast: false
182191
matrix:
183-
runs-on:
184-
[
185-
ubuntu-22.04,
186-
ubuntu-24.04,
187-
ubuntu-22.04-arm,
188-
ubuntu-24.04-arm,
189-
macos-13,
190-
macos-14,
191-
macos-15,
192-
windows-2022,
193-
windows-2025,
194-
]
192+
runs-on: [ubuntu-22.04, ubuntu-22.04-arm, macos-15, windows-2025]
195193
uses: munich-quantum-toolkit/workflows/.github/workflows/[email protected]
196194
with:
197195
runs-on: ${{ matrix.runs-on }}
198196
setup-z3: true
199197

200-
python-coverage:
201-
name: 🐍 Coverage
202-
needs: [change-detection, python-tests, python-tests-extensive]
203-
if: fromJSON(needs.change-detection.outputs.run-python-tests) && ${{ always() && !failure() && !cancelled() }}
204-
uses: munich-quantum-toolkit/workflows/.github/workflows/[email protected]
205-
permissions:
206-
contents: read
207-
id-token: write
208-
209198
python-linter:
210199
name: 🐍 Lint
211200
needs: change-detection
@@ -293,17 +282,13 @@ jobs:
293282
&& '' || 'cpp-linter,'
294283
}}
295284
${{
296-
fromJSON(needs.change-detection.outputs.run-python-tests) && !contains(github.event.pull_request.labels.*.name, 'extensive-python-ci')
297-
&& '' || 'python-tests,'
285+
fromJSON(needs.change-detection.outputs.run-python-tests)
286+
&& '' || 'python-tests,python-coverage,'
298287
}}
299288
${{
300289
fromJSON(needs.change-detection.outputs.run-python-tests) && github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'extensive-python-ci')
301290
&& '' || 'python-tests-extensive,'
302291
}}
303-
${{
304-
fromJSON(needs.change-detection.outputs.run-python-tests)
305-
&& '' || 'python-coverage,'
306-
}}
307292
${{
308293
fromJSON(needs.change-detection.outputs.run-python-tests)
309294
&& '' || 'python-linter,'

0 commit comments

Comments
 (0)