Skip to content

Commit cf0562c

Browse files
jakkdlwebknjaz
andauthored
dont use matrix for conditional variables in CI (#1357)
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
1 parent 57fd9d8 commit cf0562c

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

.github/workflows/ci-cd.yml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -41,23 +41,20 @@ jobs:
4141
os:
4242
- ubuntu-24.04
4343
- ubuntu-24.04-arm
44-
include:
45-
- experimental: false
46-
- upload-coverage: false
47-
- upload-coverage: true
48-
python-version: 3.11
49-
os: ubuntu-24.04
44+
experimental:
45+
- false
5046
fail-fast: false
5147
uses: ./.github/workflows/reusable-test.yml
5248
with:
5349
python-version: ${{ matrix.python-version }}
5450
os: ${{ matrix.os }}
5551
continue-on-error: ${{ matrix.experimental }}
5652
enable-cache: ${{ github.ref_type == 'tag' && 'false' || 'auto' }}
57-
upload-coverage: ${{ matrix.upload-coverage }}
53+
upload-coverage:
54+
${{ matrix.python-version == 3.11 && matrix.os == 'ubuntu-24.04' }}
5855
secrets:
5956
# yamllint disable-line rule:line-length
60-
codecov-token: ${{ matrix.upload-coverage && secrets.CODECOV_TOKEN || '' }}
57+
codecov-token: ${{ secrets.CODECOV_TOKEN }}
6158

6259
zizmor:
6360
name: 🌈 zizmor

0 commit comments

Comments
 (0)