Skip to content

Commit

Permalink
Merge pull request #1989 from EliahKagan/ci-cleanup
Browse files Browse the repository at this point in the history
Do some CI cleanup to make reports clearer and future changes easier
  • Loading branch information
Byron authored Jan 4, 2025
2 parents cc1c643 + 73ddb22 commit a7c7433
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 15 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/alpine-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: test-alpine
on: [push, pull_request, workflow_dispatch]

jobs:
build:
test:
runs-on: ubuntu-latest

container:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/cygwin-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: test-cygwin
on: [push, pull_request, workflow_dispatch]

jobs:
build:
test:
runs-on: windows-latest

strategy:
Expand All @@ -30,7 +30,7 @@ jobs:
- name: Set up Cygwin
uses: egor-tensin/setup-cygwin@v4
with:
packages: python39=3.9.16-1 python39-pip python39-virtualenv git
packages: python39 python39-pip python39-virtualenv git

- name: Arrange for verbose output
run: |
Expand Down
25 changes: 13 additions & 12 deletions .github/workflows/pythonpackage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,23 +9,24 @@ permissions:
contents: read

jobs:
build:
test:
strategy:
matrix:
os: [ubuntu-latest, macos-latest, windows-latest]
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
include:
- experimental: false
- os: ubuntu-22.04
os-type: [ubuntu, macos, windows]
python-version: ["3.7", "3.8", "3.9", "3.10", "3.11", "3.12"]
exclude:
- os-type: macos
python-version: "3.7"
experimental: false
- os: windows-latest
include:
- os-ver: latest
- os-type: ubuntu
python-version: "3.7"
experimental: false
os-ver: "22.04"
- experimental: false

fail-fast: false

runs-on: ${{ matrix.os }}
runs-on: ${{ matrix.os-type }}-${{ matrix.os-ver }}

defaults:
run:
Expand All @@ -43,7 +44,7 @@ jobs:
allow-prereleases: ${{ matrix.experimental }}

- name: Set up WSL (Windows)
if: startsWith(matrix.os, 'windows')
if: matrix.os-type == 'windows'
uses: Vampire/[email protected]
with:
distribution: Alpine
Expand Down Expand Up @@ -80,7 +81,7 @@ jobs:
# For debugging hook tests on native Windows systems that may have WSL.
- name: Show bash.exe candidates (Windows)
if: startsWith(matrix.os, 'windows')
if: matrix.os-type == 'windows'
run: |
set +e
bash.exe -c 'printenv WSL_DISTRO_NAME; uname -a'
Expand Down

0 comments on commit a7c7433

Please sign in to comment.