Skip to content

Commit 2a25fe1

Browse files
committed
ci: fix uv usage to use venv instead of --system flag
setup-uv creates a venv; use `uv venv` + `uv pip install` (no --system) and invoke binaries directly via .venv/bin/ as vtk-index does.
1 parent 0bcbf94 commit 2a25fe1

1 file changed

Lines changed: 18 additions & 13 deletions

File tree

.github/workflows/ci.yml

Lines changed: 18 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,21 +20,22 @@ jobs:
2020
- uses: actions/checkout@v4
2121

2222
- uses: astral-sh/setup-uv@v5
23-
with:
24-
python-version: "3.12"
23+
24+
- name: Create virtualenv
25+
run: uv venv
2526

2627
- name: Install dependencies
2728
run: |
28-
uv pip install --system \
29+
uv pip install \
2930
"git+https://github.com/vicentebolea/vtk-knowledge" \
3031
"git+https://github.com/vicentebolea/vtk-validate"
31-
uv pip install --system -e ".[dev]"
32+
uv pip install -e ".[dev]"
3233
3334
- name: ruff lint
34-
run: ruff check src/
35+
run: .venv/bin/ruff check src/
3536

3637
- name: ruff format check
37-
run: ruff format --check src/
38+
run: .venv/bin/ruff format --check src/
3839

3940
actionlint:
4041
runs-on: ubuntu-latest
@@ -61,24 +62,28 @@ jobs:
6162
- uses: actions/checkout@v4
6263

6364
- uses: astral-sh/setup-uv@v5
64-
with:
65-
python-version: ${{ matrix.python-version }}
65+
66+
- name: Set up Python ${{ matrix.python-version }}
67+
run: uv python install ${{ matrix.python-version }}
68+
69+
- name: Create virtualenv
70+
run: uv venv --python ${{ matrix.python-version }}
6671

6772
- name: Install dependencies
6873
run: |
69-
uv pip install --system \
74+
uv pip install \
7075
"git+https://github.com/vicentebolea/vtk-knowledge" \
7176
"git+https://github.com/vicentebolea/vtk-validate"
72-
uv pip install --system -e ".[dev]"
77+
uv pip install -e ".[dev]"
7378
7479
- name: Run unit tests
75-
run: pytest -m unit -v
80+
run: .venv/bin/pytest -m unit -v
7681

7782
- name: Run client tests
78-
run: pytest tests/test_client_no_server.py -v
83+
run: .venv/bin/pytest tests/test_client_no_server.py -v
7984

8085
- name: Run integration tests
81-
run: pytest -m integration -v
86+
run: .venv/bin/pytest -m integration -v
8287

8388
docker-deploy:
8489
name: Build and Push Deployment Image

0 commit comments

Comments
 (0)