Skip to content

Commit

Permalink
Remove matrix strategy from GitHub actions
Browse files Browse the repository at this point in the history
I think the usefulness of this is limited: most tools can probably
migrate from one Python version to another in a single commit, and
replace the Python version in CI at the same time. Let’s simplify the
setup for that common case. (Part of the motivation is that matrix runs
are a bit more annoying to navigate on GitHub, even when there’s only
one instance of them.)
  • Loading branch information
lucaswerkmeister committed Jul 15, 2023
1 parent c81fe4c commit 0d7b2d7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 15 deletions.
12 changes: 2 additions & 10 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,11 @@ on:
jobs:
test-defaults:
runs-on: ubuntu-latest
strategy:
matrix:
python-version:
- '3.11'
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}
python-version: '3.11'
- run: pip install --upgrade pip cookiecutter
- run: |
cd "${TMPDIR:-/tmp}"
Expand All @@ -24,15 +20,11 @@ jobs:
python example-tool/app.py
test-all:
runs-on: ubuntu-latest
strategy:
matrix:
python-version:
- '3.11'
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}
python-version: '3.11'
- run: pip install --upgrade pip cookiecutter
- run: |
cd "${TMPDIR:-/tmp}"
Expand Down
6 changes: 1 addition & 5 deletions {{cookiecutter.tool_identifier}}/.github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,11 @@ on:
jobs:
test:
runs-on: ubuntu-latest
strategy:
matrix:
python-version:
- '3.11'
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: {% raw %}${{ matrix.python-version }}{% endraw %}
python-version: '3.11'
- run: pip install --upgrade pip
- run: pip install -r requirements.txt
- run: make check

0 comments on commit 0d7b2d7

Please sign in to comment.