Skip to content

Commit 63363bf

Browse files
fix: use proper dependency resolution in CI
The CI was incorrectly using --frozen with both resolution strategies. The --frozen flag should only be used with the highest resolution strategy (which uses the lockfile), while lowest-direct resolution needs to resolve dependencies dynamically using --resolution lowest-direct. This change restructures the matrix to properly pass the correct flags for each resolution strategy, fixing the CI pipeline. Github-Issue: #1325
1 parent 35a9ccd commit 63363bf

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

.github/workflows/shared.yml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,13 +30,18 @@ jobs:
3030
SKIP: no-commit-to-branch
3131

3232
test:
33+
name: Test (${{ matrix.python-version }}) (${{ matrix.os }}) (${{ matrix.dep-resolution.name }})
3334
runs-on: ${{ matrix.os }}
3435
timeout-minutes: 10
3536
continue-on-error: true
3637
strategy:
3738
matrix:
3839
python-version: ["3.10", "3.11", "3.12", "3.13"]
39-
dep-resolution: ["lowest-direct", "highest"]
40+
dep-resolution:
41+
- name: lowest-direct
42+
install-flags: "--resolution lowest-direct"
43+
- name: highest
44+
install-flags: "--frozen"
4045
os: [ubuntu-latest, windows-latest]
4146

4247
steps:
@@ -49,7 +54,7 @@ jobs:
4954
version: 0.9.5
5055

5156
- name: Install the project
52-
run: uv sync --frozen --all-extras --python ${{ matrix.python-version }} --resolution ${{ matrix.dep-resolution }}
57+
run: uv sync ${{ matrix.dep-resolution.install-flags }} --all-extras --python ${{ matrix.python-version }}
5358

5459
- name: Run pytest
5560
run: uv run --frozen --no-sync pytest

0 commit comments

Comments
 (0)