Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TST: never compile dependencies when testing #238

Merged
merged 1 commit into from
Aug 6, 2024
Merged

TST: never compile dependencies when testing #238

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 11 additions & 10 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,17 +80,17 @@ jobs:

- if: matrix.deps == 'minimal'
run: |
uv pip compile pyproject.toml --resolution=lowest-direct | uv pip install -r -
uv pip compile pyproject.toml --resolution=lowest-direct | uv pip install -r - --no-build

- name: Build library
env:
GPGI_PY_LIB: ${{ matrix.marker == 'PY_LIB' }}
# install in editable mode to allow coverage measurement
run: |
echo GPGI_PY_LIB=$GPGI_PY_LIB
uv pip install --editable .
uv pip install --editable . --no-build

- run: uv pip install --requirement requirements/tests.txt
- run: uv pip install --requirement requirements/tests.txt --no-build

- run: uv pip list

Expand Down Expand Up @@ -125,9 +125,9 @@ jobs:
uv-venv: .venv

- name: Build library
run: uv pip install .
run: uv pip install . --only-binary numpy

- run: uv pip install --requirement requirements/tests.txt
- run: uv pip install --requirement requirements/tests.txt --no-build

- name: Run Image Tests
run: |
Expand Down Expand Up @@ -177,9 +177,9 @@ jobs:
uv-venv: .venv

- name: Build library
run: uv pip install .
run: uv pip install . --only-binary numpy

- run: uv pip install --requirement requirements/tests.txt
- run: uv pip install --requirement requirements/tests.txt --no-build

- name: Run Concurrency Tests
run: |
Expand All @@ -202,7 +202,7 @@ jobs:
uv-version: 0.2.33
uv-venv: .venv

- run: uv pip install 'coverage[toml]'
- run: uv pip install 'coverage[toml]' --no-build

- uses: actions/download-artifact@v4
with:
Expand Down Expand Up @@ -247,7 +247,7 @@ jobs:
uv-venv: .venv

- name: Build
run: uv pip install --requirement requirements/typecheck.txt
run: uv pip install --requirement requirements/typecheck.txt --no-build

- name: Typecheck
run: |
Expand Down Expand Up @@ -289,9 +289,10 @@ jobs:
python -m pip install -U pip
python -m pip install --pre setuptools wheel
python -m pip install --pre numpy Cython \
--only-binary ':all:' \
--extra-index-url \
https://pypi.anaconda.org/scientific-python-nightly-wheels/simple
python -m pip install . --no-build-isolation
python -m pip install . --no-build-isolation --no-deps

- name: Install minimal test env
run: python -m pip install --pre pytest pytest-repeat
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ build-frontend = "build[uv]"
build-verbosity = 1
test-skip = "*-musllinux*"
test-command = [
"uv pip install -r {project}/requirements/tests.txt",
"uv pip install -r {project}/requirements/tests.txt --no-build",
"pytest --color=yes -ra {project}",
"pytest --color=yes --count 500 {project}/tests/test_concurrent.py",
]
Expand Down