Skip to content

Commit b124709

Browse files
committed
fix: use virtual environment in CI instead of system packages
- uv-managed Python environments don't allow --system installs - Create and activate virtual environment before installing packages - Update all CI steps to use the virtual environment
1 parent 78251a6 commit b124709

1 file changed

Lines changed: 15 additions & 5 deletions

File tree

.github/workflows/build-reusable.yml

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -201,16 +201,20 @@ jobs:
201201
202202
- name: Install built packages for testing
203203
run: |
204+
# Create a virtual environment
205+
uv venv
206+
source .venv/bin/activate || source .venv/Scripts/activate
207+
204208
# Install the built wheels
205209
if [[ "${{ matrix.os }}" == ubuntu-* ]]; then
206-
uv pip install --system packages/leann-core/dist/*.whl
207-
uv pip install --system packages/leann/dist/*.whl
210+
uv pip install packages/leann-core/dist/*.whl
211+
uv pip install packages/leann/dist/*.whl
208212
fi
209-
uv pip install --system packages/leann-backend-hnsw/dist/*.whl
210-
uv pip install --system packages/leann-backend-diskann/dist/*.whl
213+
uv pip install packages/leann-backend-hnsw/dist/*.whl
214+
uv pip install packages/leann-backend-diskann/dist/*.whl
211215
212216
# Install test dependencies using extras
213-
uv pip install --system -e ".[test]"
217+
uv pip install -e ".[test]"
214218
215219
- name: Run tests with pytest
216220
env:
@@ -220,11 +224,17 @@ jobs:
220224
TOKENIZERS_PARALLELISM: false
221225
PYTORCH_ENABLE_MPS_FALLBACK: 0 # Disable MPS on macOS CI to avoid memory issues
222226
run: |
227+
# Activate virtual environment
228+
source .venv/bin/activate || source .venv/Scripts/activate
229+
223230
# Run all tests
224231
pytest tests/
225232
226233
- name: Run sanity checks (optional)
227234
run: |
235+
# Activate virtual environment
236+
source .venv/bin/activate || source .venv/Scripts/activate
237+
228238
# Run distance function tests if available
229239
if [ -f test/sanity_checks/test_distance_functions.py ]; then
230240
echo "Running distance function sanity checks..."

0 commit comments

Comments
 (0)