Skip to content

Commit

Permalink
Cython no longer needs a matrix in CI workflows
Browse files Browse the repository at this point in the history
  • Loading branch information
berquist committed Apr 2, 2024
1 parent 2198407 commit a1eed4d
Showing 1 changed file with 1 addition and 20 deletions.
21 changes: 1 addition & 20 deletions .github/workflows/CI.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on: [push, pull_request]

jobs:
test:
name: Test ${{ matrix.os }} / ${{ matrix.python-version }} / Cython ${{ matrix.cythonize }} / ${{ matrix.env-type.env-type-name }}
name: Test ${{ matrix.os }} / ${{ matrix.python-version }} / ${{ matrix.env-type.env-type-name }}
strategy:
matrix:
os:
Expand All @@ -20,21 +20,6 @@ jobs:
- { env-type-name: venv, shell: bash }
- { env-type-name: conda, shell: 'bash -l {0}' }
# yamllint enable rule:braces
cythonize: [true, false]
# Skip some overkill jobs.
exclude:
- os: ubuntu-latest
env-type: { env-type-name: venv, shell: bash }
cythonize: false
- os: ubuntu-latest
env-type: { env-type-name: conda, shell: 'bash -l {0}' }
cythonize: true
- os: macos-latest
env-type: { env-type-name: conda, shell: 'bash -l {0}' }
cythonize: false
- os: macos-latest
env-type: { env-type-name: venv, shell: bash }
cythonize: true
runs-on: ${{ matrix.os }}
defaults:
run:
Expand All @@ -58,10 +43,6 @@ jobs:
if: ${{ matrix.env-type.env-type-name == 'conda' }}
- name: Update pip and setuptools
run: python -m pip install -U pip setuptools
- name: Install Cython
run: |
python -m pip install 'Cython<3'
if: ${{ matrix.cythonize }}
- name: Print pip config
run: python -m pip config list
- name: Install test dependencies
Expand Down

0 comments on commit a1eed4d

Please sign in to comment.