Skip to content

Commit

Permalink
CI: fail with warnings, and several minor tunings to test workflow (#325
Browse files Browse the repository at this point in the history
)
  • Loading branch information
mwtoews authored Jul 11, 2024
1 parent d42650c commit fa284a9
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 27 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:

- uses: actions/upload-artifact@v4
with:
name: cibw-wheels-${{ matrix.os }}-${{ strategy.job-index }}
name: cibw-wheels-${{ matrix.os }}
path: ./wheelhouse/*.whl

build_sdist:
Expand Down
42 changes: 16 additions & 26 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,33 +4,24 @@ on:
push:
branches:
- master
paths:
- '.github/workflows/test.yml'
pull_request:
workflow_dispatch:
schedule:
- cron: '0 6 * * 1'

jobs:
pre-commit:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
- uses: pre-commit/[email protected]

conda:
name: Conda ${{ matrix.python-version }} - ${{ matrix.os }}
defaults:
run:
shell: bash -l {0}
runs-on: ${{ matrix.os }}
strategy:
fail-fast: true
fail-fast: false
matrix:
os: ['ubuntu-latest', 'macos-latest', 'windows-latest']
python-version: ['3.9', '3.10', '3.11', '3.12']
# test oldesst and newest libspatialindex versions
# test oldest and newest libspatialindex versions
sidx-version: ['1.8.5', '2.0.0']
exclude:
- os: 'macos-latest'
Expand All @@ -43,16 +34,15 @@ jobs:
channels: conda-forge
auto-update-conda: true
python-version: ${{ matrix.python-version }}

- name: Setup
run: |
conda install -c conda-forge numpy libspatialindex=${{ matrix.sidx-version }} -y
run: conda install -c conda-forge numpy pytest libspatialindex=${{ matrix.sidx-version }} -y

- name: Install
run: |
pip install -e .
run: pip install -e .

- name: Test with pytest
run: |
pip install pytest
python -m pytest --doctest-modules rtree tests
run: pytest --import-mode=importlib -Werror -v --doctest-modules rtree tests

ubuntu:
name: Ubuntu Python ${{ matrix.python-version }}
Expand All @@ -61,7 +51,7 @@ jobs:
shell: bash -l {0}
runs-on: ubuntu-latest
strategy:
fail-fast: true
fail-fast: false
matrix:
python-version: ['3.9', '3.10', '3.11', '3.12', '3.13']

Expand All @@ -72,15 +62,15 @@ jobs:
with:
python-version: ${{ matrix.python-version }}
allow-prereleases: true

- name: Setup
run: |
sudo apt install libspatialindex-c6 python3-pip
python3 -m pip install --upgrade pip
python3 -m pip install setuptools numpy pytest
sudo apt-get -y install libspatialindex-c6
pip install --upgrade pip
pip install numpy pytest
- name: Build
run: |
python3 -m pip install --user .
run: pip install --user .

- name: Test with pytest
run: |
python3 -m pytest --doctest-modules rtree tests
run: pytest --import-mode=importlib -Werror -v --doctest-modules rtree tests

0 comments on commit fa284a9

Please sign in to comment.