Skip to content

Commit

Permalink
New Github Action for testing imports (python-graphblas#357)
Browse files Browse the repository at this point in the history
* New Github Action for testing imports

* Specify Python version here too
  • Loading branch information
eriknw authored Dec 31, 2022
1 parent e388cb8 commit 5cb766e
Show file tree
Hide file tree
Showing 3 changed files with 24 additions and 4 deletions.
22 changes: 22 additions & 0 deletions .github/workflows/imports.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
name: Test imports

on:
pull_request:
push:
branches:
- main

jobs:
test_imports:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ["3.8", "3.9", "3.10"]
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}
- run: python -m pip install --upgrade pip
- run: pip install -e .
- run: ./scripts/test_imports.sh
2 changes: 2 additions & 0 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,4 +16,6 @@ jobs:
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: "3.10"
- uses: pre-commit/[email protected]
4 changes: 0 additions & 4 deletions .github/workflows/test_and_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,10 +83,6 @@ jobs:
mamba install -c conda-forge python-suitesparse-graphblas
fi
python setup.py develop --no-deps
- name: Verify build
run: |
# Make sure all top-level imports work
./scripts/test_imports.sh
- name: Unit tests
run: |
coverage run --branch -m pytest ${{ matrix.cfg.testopts }} -v
Expand Down

0 comments on commit 5cb766e

Please sign in to comment.