TBNilles/LEANN
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use virtual environment in CI instead of system packages

Browse Source 
- 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
This commit is contained in:
Andy Lee
2025-07-28 16:04:49 -07:00
parent 78251a6d4c
commit b124709bcd
1 changed files with 15 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
.github/workflows/build-reusable.yml vendored
View File

@@ -201,16 +201,20 @@ jobs:
- name: Install built packages for testing
run: |
# Create a virtual environment
uv venv
source .venv/bin/activate || source .venv/Scripts/activate
# Install the built wheels
if [[ "${{ matrix.os }}" == ubuntu-* ]]; then
uv pip install --system packages/leann-core/dist/*.whl
uv pip install --system packages/leann/dist/*.whl
uv pip install packages/leann-core/dist/*.whl
uv pip install packages/leann/dist/*.whl
fi
uv pip install --system packages/leann-backend-hnsw/dist/*.whl
uv pip install --system packages/leann-backend-diskann/dist/*.whl
uv pip install packages/leann-backend-hnsw/dist/*.whl
uv pip install packages/leann-backend-diskann/dist/*.whl
# Install test dependencies using extras
uv pip install --system -e ".[test]"
uv pip install -e ".[test]"
- name: Run tests with pytest
env:
@@ -220,11 +224,17 @@ jobs:
TOKENIZERS_PARALLELISM: false
PYTORCH_ENABLE_MPS_FALLBACK: 0 # Disable MPS on macOS CI to avoid memory issues
run: |
# Activate virtual environment
source .venv/bin/activate || source .venv/Scripts/activate
# Run all tests
pytest tests/
- name: Run sanity checks (optional)
run: |
# Activate virtual environment
source .venv/bin/activate || source .venv/Scripts/activate
# Run distance function tests if available
if [ -f test/sanity_checks/test_distance_functions.py ]; then
echo "Running distance function sanity checks..."