Skip to content

Commit ae4c2a4

Browse files
ci: note the Windows pip leg resolves a different test dep-set than pixi
The pip fallback resolves [project.optional-dependencies].test, which differs from the pixi legs' [dependency-groups].test (e.g. omits graphviz). Flag the latent trap so a dep-set mismatch isn't mistaken for an OS-specific failure. Addresses MilagrosMarin's review on #1522.
1 parent 0e9eebb commit ae4c2a4

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

.github/workflows/test.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,12 @@ jobs:
9292
with:
9393
python-version: ${{ matrix.python-version }}
9494

95+
# NOTE: pip resolves `[project.optional-dependencies].test`, which is a
96+
# DIFFERENT set than the pixi Linux legs' `[dependency-groups].test`
97+
# (e.g. it omits graphviz). A unit test importing a dep present in only
98+
# one set would then pass on Linux but error on Windows (or vice versa) —
99+
# a dep-set mismatch that reads like an OS bug. Keep the two `test` sets
100+
# in sync when unit-test dependencies change.
95101
- name: Install package with test extras
96102
run: pip install -e ".[test]"
97103

0 commit comments

Comments
 (0)