Skip to content

Commit 53f810d

Browse files
mrdavearmsclaude
andcommitted
ci: run the test suite on Windows and macOS too
The app ships on Windows and macOS but CI only ran on Linux. Add a windows-latest + macos-latest matrix job (kept separate from the 'test' job so the branch-protection check name is preserved) for real cross-platform regression coverage. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 7d90a38 commit 53f810d

1 file changed

Lines changed: 27 additions & 0 deletions

File tree

.github/workflows/test.yml

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ on:
77
branches: [main]
88

99
jobs:
10+
# Linux job keeps the name `test` so the branch-protection required
11+
# status check on `main` continues to match.
1012
test:
1113
runs-on: ubuntu-latest
1214
steps:
@@ -24,3 +26,28 @@ jobs:
2426
2527
- name: Run tests
2628
run: python -m pytest tests/ -v
29+
30+
# Windows + macOS coverage — the app ships on both, so the suite must
31+
# pass there too. Separate jobs (not a matrix) so the `test` check name
32+
# above is preserved for branch protection.
33+
test-cross-platform:
34+
strategy:
35+
fail-fast: false
36+
matrix:
37+
os: [windows-latest, macos-latest]
38+
runs-on: ${{ matrix.os }}
39+
steps:
40+
- uses: actions/checkout@v6
41+
42+
- uses: actions/setup-python@v6
43+
with:
44+
python-version: '3.10'
45+
46+
- name: Install dependencies
47+
run: |
48+
python -m pip install --upgrade pip
49+
pip install -r requirements.txt
50+
pip install pytest
51+
52+
- name: Run tests
53+
run: python -m pytest tests/ -v

0 commit comments

Comments
 (0)