Skip to content

Commit f19ec4f

Browse files
gijzelaerrclaude
andcommitted
Add uv venv back - uv pip requires existing venv
uv pip install doesn't auto-create venv, so we need explicit uv venv before uv pip install. Still simplified by using default .venv path and uv run for execution. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 78754f9 commit f19ec4f

7 files changed

Lines changed: 16 additions & 5 deletions

.github/workflows/linux-build-test-amd64.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ jobs:
6060

6161
- name: Install python-snap7
6262
run: |
63+
uv venv
6364
uv pip install pytest
6465
uv pip install dist/*.whl
6566

.github/workflows/linux-build-test-arm64.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ jobs:
6565
--platform linux/arm64 \
6666
"arm64v8/python:${{ matrix.python-version }}-bookworm" /bin/bash -s <<EOF
6767
pip install --upgrade pip uv
68+
uv venv
6869
uv pip install pytest
6970
uv pip install dist/*.whl
7071
uv run pytest -m "server or util or client or mainloop or partner"

.github/workflows/linux-test-with-deb.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ jobs:
2828
- name: Install uv
2929
uses: astral-sh/setup-uv@v5
3030
- name: Install dependencies
31-
run: uv pip install ".[test]"
31+
run: |
32+
uv venv
33+
uv pip install ".[test]"
3234
- name: Run pytest
3335
run: |
3436
uv run pytest -m "server or util or client or mainloop"

.github/workflows/osx-build-test-amd64.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ jobs:
7171

7272
- name: Install python-snap7
7373
run: |
74+
uv venv
7475
uv pip install pytest
7576
uv pip install dist/*.whl
7677

.github/workflows/osx-test-with-brew.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,10 @@ jobs:
2020
- name: Install uv
2121
uses: astral-sh/setup-uv@v5
2222
- name: Install python-snap7
23-
run: uv pip install --python python${{ matrix.python-version }} -e ".[test]"
23+
run: |
24+
uv venv --python python${{ matrix.python-version }}
25+
uv pip install -e ".[test]"
2426
- name: Run pytest
2527
run: |
26-
uv run --python python${{ matrix.python-version }} pytest -m "server or util or client or mainloop"
28+
uv run pytest -m "server or util or client or mainloop"
2729
sudo .venv/bin/pytest -m partner

.github/workflows/publish-pypi.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,9 @@ jobs:
6868
uses: astral-sh/setup-uv@v5
6969

7070
- name: install python-snap7
71-
run: uv pip install --extra-index-url https://test.pypi.org/simple/ python-snap7[test]
71+
run: |
72+
uv venv
73+
uv pip install --extra-index-url https://test.pypi.org/simple/ python-snap7[test]
7274
7375
test-pypi-package-windows:
7476
runs-on: ${{ matrix.os }}

.github/workflows/publish-test-pypi.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,9 @@ jobs:
7171
uses: astral-sh/setup-uv@v5
7272

7373
- name: install python-snap7
74-
run: uv pip install --extra-index-url https://test.pypi.org/simple/ python-snap7[test]
74+
run: |
75+
uv venv
76+
uv pip install --extra-index-url https://test.pypi.org/simple/ python-snap7[test]
7577
7678
test-pypi-package-windows:
7779
runs-on: ${{ matrix.os }}

0 commit comments

Comments
 (0)