Skip to content

ci: reduce runner costs — Linux-first matrix with targeted cross-platform spot-checks #25

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
harell wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ci: reduce runner costs — Linux-first matrix with targeted cross-platform spot-checks #25

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 18 additions & 10 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,6 @@ jobs:
with:
persist-credentials: false

- name: Setup git user config
run: |
git config --global user.name placeholder
git config --global user.email placeholder@example.com

- name: Set up uv
uses: astral-sh/setup-uv@f94ec6bedd8674c4426838e6b50417d36b6ab231 # v5.3.1
with:
Expand All @@ -48,10 +43,10 @@ jobs:
env:
PYTHON_VERSION: ${{ matrix.python-version }}

- name: Run pre-commit
- name: Run Prek
if: matrix.checks
run: |
uv run pre-commit run --all-files
uv run --frozen prek run --all-files

- name: Run pyright
if: matrix.checks
Expand All @@ -75,29 +70,41 @@ jobs:
MPLBACKEND: Agg # https://github.com/orgs/community/discussions/26434

- name: Create test reports directory
if: matrix.pytest && matrix.os == 'ubuntu-latest' && matrix.python-version == '3.10' && matrix.resolution == 'highest'
if: matrix.pytest && matrix.python-version == '3.10' && matrix.resolution == 'highest'
run: mkdir -p ./test-reports

- name: Upload coverage reports
if: matrix.pytest && matrix.os == 'ubuntu-latest' && matrix.python-version == '3.10' && matrix.resolution == 'highest'
if: matrix.pytest && matrix.python-version == '3.10' && matrix.resolution == 'highest'
uses: actions/upload-artifact@b4b15b8c7c6ac21ea08fcf65892d2ee8f75cf882 # v4.4.3
with:
name: coverage-reports
path: test-reports/coverage.xml

strategy:
matrix:
os: ["ubuntu-latest", "macos-latest", "windows-latest"]
os: ["ubuntu-latest"]
python-version: ["3.10", "3.11", "3.12", "3.13"]
resolution: ["highest"]
checks: [true]
pytest: [true]
include:
# Min-dependency run on Linux
- os: "ubuntu-latest"
python-version: "3.10"
resolution: "lowest-direct"
checks: false
pytest: true
# Cross-platform spot-checks on latest Python only (cost-reduced from full matrix)
- os: "windows-latest"
python-version: "3.13"
resolution: "highest"
checks: false
pytest: true
- os: "macos-latest"
python-version: "3.13"
resolution: "highest"
checks: false
pytest: true

code-analysis:
name: Analyse Code Quality
Expand Down Expand Up @@ -141,3 +148,4 @@ jobs:
env:
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
SONAR_HOST_URL: ${{ vars.SONAR_HOST_URL }}

Loading