Skip to content

Commit 680017e

Browse files
committed
fix(ci): explicitly install maturin before running maturin develop
The 'uv sync' command installs project dependencies but not build-system requirements like maturin. Added explicit 'uv pip install maturin' step before attempting to run 'uv run maturin develop'. Related to #22
1 parent dcd28ca commit 680017e

File tree

1 file changed

+16
-8
lines changed

1 file changed

+16
-8
lines changed

.github/workflows/ci.yml

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -56,10 +56,14 @@ jobs:
5656
- name: Install uv
5757
uses: astral-sh/setup-uv@v5
5858

59-
- name: Install dependencies and build
60-
run: |
61-
uv sync --all-extras
62-
uv run maturin develop
59+
- name: Install dependencies
60+
run: uv sync --all-extras
61+
62+
- name: Install maturin
63+
run: uv pip install maturin
64+
65+
- name: Build Rust extension
66+
run: uv run maturin develop
6367

6468
- name: Run tests
6569
run: uv run pytest
@@ -83,10 +87,14 @@ jobs:
8387
- name: Install uv
8488
uses: astral-sh/setup-uv@v5
8589

86-
- name: Install dependencies and build
87-
run: |
88-
uv sync --all-extras
89-
uv run maturin develop
90+
- name: Install dependencies
91+
run: uv sync --all-extras
92+
93+
- name: Install maturin
94+
run: uv pip install maturin
95+
96+
- name: Build Rust extension
97+
run: uv run maturin develop
9098

9199
- name: Run tests with coverage
92100
run: uv run pytest --cov=dioxide --cov-report=xml --cov-report=term-missing

0 commit comments

Comments
 (0)