Skip to content

Commit 5ec3877

Browse files
committed
ci: express "no x86 for pre-3.11 Python" as a single matrix policy
The previous commit listed three explicit (os, python, architecture) triples to exclude. Since x86 is already excluded on ubuntu-latest and macos-latest by their own (os, architecture) rules, the os qualifier on these entries is redundant. Replace the triples with (python, architecture) pairs so the policy reads as "no x86 testing for Python below 3.11" — which is the actual rule we care about. cp311-abi3-win32 wheels from `cryptography` cover 3.11+, so those cells remain in the matrix. No change in effective coverage.
1 parent a3d4248 commit 5ec3877

1 file changed

Lines changed: 5 additions & 9 deletions

File tree

‎.github/workflows/ci_tests.yaml‎

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -73,17 +73,13 @@ jobs:
7373
# (Rust/PyO3). That build fails on the hosted windows-latest
7474
# runner because only the x86_64 Rust target is installed while
7575
# the interpreter reports platform `win32` (i686), so maturin
76-
# refuses to use it. The job is therefore unbuildable on
77-
# GitHub-hosted runners until a host-side toolchain workaround
78-
# exists; drop these combinations from the matrix.
79-
- os: windows-latest
80-
python: 3.8
76+
# refuses to use it. Until a host-side toolchain workaround
77+
# exists, omit pre-3.11 Python from x86 testing entirely.
78+
- python: 3.8
8179
architecture: x86
82-
- os: windows-latest
83-
python: 3.9
80+
- python: 3.9
8481
architecture: x86
85-
- os: windows-latest
86-
python: "3.10"
82+
- python: "3.10"
8783
architecture: x86
8884

8985
# MPL is not supported on <3.11

0 commit comments

Comments
 (0)