Skip to content

Commit

Permalink
[build] Run tests on Windows.
Browse files Browse the repository at this point in the history
The use of temporary files for injecting fixtures into packages shows platform-specific behavior. As a result, it makes sense to run the tests on Windows as well, at least as long as the virtual module workaround is in place.

Signed-off-by: Michael Seifert <m.seifert@digitalernachschub.de>
  • Loading branch information
seifertm committed Jan 10, 2024
1 parent aecfd01 commit ef619b9
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,12 @@ jobs:
path: dist

test:
name: Python ${{ matrix.python-version }}
runs-on: ubuntu-latest
name: ${{ matrix.os }} - Python ${{ matrix.python-version }}
runs-on: ${{ matrix.os }}

strategy:
matrix:
os: [ubuntu-latest, windows-latest]
python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']

steps:
Expand All @@ -75,7 +76,6 @@ jobs:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: |
set -xe
python -VV
python -m site
python -m pip install --upgrade pip
Expand All @@ -85,6 +85,7 @@ jobs:

- name: Store coverage data
uses: actions/upload-artifact@v3
if: "!endsWith(matrix.os, 'windows')"
with:
name: coverage-per-interpreter
path: .coverage.*
Expand Down

0 comments on commit ef619b9

Please sign in to comment.