Skip to content

Commit 8f3ffeb

Browse files
committed
fix presubmit
1 parent acff3ec commit 8f3ffeb

1 file changed

Lines changed: 13 additions & 4 deletions

File tree

.github/workflows/unittest.yml

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,17 @@ permissions:
1616
env:
1717
BATCH_SIZE: 5
1818
TEST_ALL_PACKAGES: "true" # Set to "false" to only run tests for packages with a git diff
19+
ALL_PYTHON: "['3.9', '3.10', '3.11', '3.12', '3.13', '3.14']"
1920

2021
jobs:
22+
# Preserve the workaround for env variables in matrix
23+
python_config:
24+
runs-on: ubuntu-latest
25+
outputs:
26+
all_python: ${{ env.ALL_PYTHON }}
27+
steps:
28+
- run: echo "Initializing test configuration matrix definitions"
29+
2130
# Dynamic package discovery job to calculate required matrix size automatically
2231
discover-packages:
2332
runs-on: ubuntu-latest
@@ -71,10 +80,10 @@ jobs:
7180
unit:
7281
name: "unit-run (${{ matrix.python }}, Batch ${{ matrix.batch-index }})"
7382
runs-on: ubuntu-22.04
74-
needs: discover-packages
83+
needs: [python_config, discover-packages]
7584
strategy:
7685
matrix:
77-
python: ['3.9', '3.10', "3.11", "3.12", "3.13", "3.14"]
86+
python: ${{ fromJSON(needs.python_config.outputs.all_python) }}
7887
# Dynamically scales to fit every package perfectly without hardcoding array indices
7988
batch-index: ${{ fromJson(needs.discover-packages.outputs.batch-indices) }}
8089
steps:
@@ -264,11 +273,11 @@ jobs:
264273
265274
unittest-runtime-result:
266275
name: "unit (${{ matrix.python }})"
267-
needs: unit
276+
needs: [python_config, unit]
268277
if: always()
269278
strategy:
270279
matrix:
271-
python: ['3.9', '3.10', "3.11", "3.12", "3.13", "3.14"]
280+
python: ${{ fromJSON(needs.python_config.outputs.all_python) }}
272281
runs-on: ubuntu-latest
273282
steps:
274283
- name: Download Python-Specific Footprints

0 commit comments

Comments
 (0)