From e200314e0c236c40e38702554a87b93efa8a7948 Mon Sep 17 00:00:00 2001 From: Chris Fuller Date: Thu, 28 May 2026 16:19:59 +0100 Subject: [PATCH 1/4] Add Windows to CI testing matrix --- .github/workflows/run_tests.yaml | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/.github/workflows/run_tests.yaml b/.github/workflows/run_tests.yaml index b95f0bd..f501813 100644 --- a/.github/workflows/run_tests.yaml +++ b/.github/workflows/run_tests.yaml @@ -10,13 +10,18 @@ on: - main - dev +defaults: + run: + shell: bash + jobs: unit_tests: - runs-on: ubuntu-latest + runs-on: ${{ matrix.os }} strategy: fail-fast: false matrix: python-version: ["3.10", "3.11", "3.12", "3.13"] + os: [ubuntu-latest, windows-latest] steps: - uses: actions/checkout@v4 @@ -34,4 +39,4 @@ jobs: run: poetry install - name: Run tests - run: sh ci/run_tests.sh + run: poetry run pytest tests From d6703260a7f60beb1b50798000b4efbdc7633f33 Mon Sep 17 00:00:00 2001 From: Chris Fuller Date: Thu, 28 May 2026 17:01:29 +0100 Subject: [PATCH 2/4] Remove unused run_tests.sh script --- ci/run_tests.sh | 4 ---- git_hooks/pre-push | 2 +- 2 files changed, 1 insertion(+), 5 deletions(-) delete mode 100644 ci/run_tests.sh diff --git a/ci/run_tests.sh b/ci/run_tests.sh deleted file mode 100644 index 036789c..0000000 --- a/ci/run_tests.sh +++ /dev/null @@ -1,4 +0,0 @@ -#!/bin/bash -set -e - -poetry run pytest tests diff --git a/git_hooks/pre-push b/git_hooks/pre-push index f3a2dec..254e819 100755 --- a/git_hooks/pre-push +++ b/git_hooks/pre-push @@ -10,4 +10,4 @@ echo "[2/3] Running linter..." sh ./ci/run_linter.sh echo "[3/3] Running unit tests..." -sh ./ci/run_tests.sh \ No newline at end of file +poetry run pytest tests \ No newline at end of file From ac8625eecff05cf331bd58f092dfd8e35caf3b11 Mon Sep 17 00:00:00 2001 From: Chris Fuller Date: Mon, 1 Jun 2026 08:57:33 +0100 Subject: [PATCH 3/4] Use bash on Linux and pwsh on Windows --- .github/workflows/run_tests.yaml | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/.github/workflows/run_tests.yaml b/.github/workflows/run_tests.yaml index f501813..0e72dc9 100644 --- a/.github/workflows/run_tests.yaml +++ b/.github/workflows/run_tests.yaml @@ -10,18 +10,22 @@ on: - main - dev -defaults: - run: - shell: bash - jobs: unit_tests: runs-on: ${{ matrix.os }} + defaults: + run: + shell: ${{ matrix.shell }} strategy: fail-fast: false matrix: python-version: ["3.10", "3.11", "3.12", "3.13"] os: [ubuntu-latest, windows-latest] + include: + - os: ubuntu-latest + shell: bash + - os: windows-latest + shell: pwsh steps: - uses: actions/checkout@v4 From f5cce6f46777cca2d36ba3b2449310bb495a7369 Mon Sep 17 00:00:00 2001 From: Chris Fuller Date: Mon, 1 Jun 2026 09:00:57 +0100 Subject: [PATCH 4/4] Remove explicit shell config, rely on runner defaults --- .github/workflows/run_tests.yaml | 8 -------- 1 file changed, 8 deletions(-) diff --git a/.github/workflows/run_tests.yaml b/.github/workflows/run_tests.yaml index 0e72dc9..949815d 100644 --- a/.github/workflows/run_tests.yaml +++ b/.github/workflows/run_tests.yaml @@ -13,19 +13,11 @@ on: jobs: unit_tests: runs-on: ${{ matrix.os }} - defaults: - run: - shell: ${{ matrix.shell }} strategy: fail-fast: false matrix: python-version: ["3.10", "3.11", "3.12", "3.13"] os: [ubuntu-latest, windows-latest] - include: - - os: ubuntu-latest - shell: bash - - os: windows-latest - shell: pwsh steps: - uses: actions/checkout@v4