Skip to content

Commit

Permalink
Yet another attempt at parametrization after realizing there was a typo
Browse files Browse the repository at this point in the history
  • Loading branch information
mstimberg committed May 31, 2021
1 parent f2a2c1d commit 989cd3f
Showing 1 changed file with 16 additions and 5 deletions.
21 changes: 16 additions & 5 deletions .github/workflows/testsuite.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,19 @@ name: TestSuite

on: [push, pull_request]

jobs:
jobs:
set_python_versions:
name: "Set Python versions"
runs-on: ubuntu-latest
outputs:
max_python: "3.9"
min_python: "3.7"
steps:
- name: "Do-nothing step"
run: echo Doing nothing

testing:
needs: [set_python_versions]
name: "Python ${{ matrix.python-version }} on ${{ matrix.os }} (standalone: ${{ matrix.standalone }}, 32bit: ${{ matrix.float_dtype_32 }})"
runs-on: ${{ matrix.os }}
strategy:
Expand All @@ -12,15 +23,15 @@ jobs:
os: [ubuntu-20.04, windows-2019, macOS-10.15]
standalone: [false, true]
float_dtype_32: [false,true]
python_version: [3.9]
python-version: [${{ needs.set_python_versions.outputs.max_python }}]
include:
- os: ubuntu-20.04
standalone: false
python-version: 3.7
python-version: ${{ needs.set_python_versions.outputs.min_python }}
float_dtype_32: false
- os: ubuntu-20.04
standalone: true
python-version: 3.7
python-version: ${{ needs.set_python_versions.outputs.min_python }}
float_dtype_32: false

defaults:
Expand Down Expand Up @@ -58,7 +69,7 @@ jobs:
FLOAT_DTYPE_32: ${{ matrix.float_dtype_32 }}

- name: Upload coverage data to coveralls
if: ${{ startsWith(matrix.os, 'ubuntu-') && matrix.python-version == 3.9 }}
if: ${{ startsWith(matrix.os, 'ubuntu-') && matrix.python-version == needs.set_python_versions.outputs.max_python }}
run: |
cp $GITHUB_WORKSPACE/../.coverage .
conda install --quiet --yes coveralls
Expand Down

0 comments on commit 989cd3f

Please sign in to comment.