From 801f0044e1587d24586715e8d7632104d7d158c5 Mon Sep 17 00:00:00 2001
From: "Christian Y. Brenninkmeijer"
 <christian.brenninkmeijer@manchester.ac.uk>
Date: Fri, 17 May 2024 08:22:46 +0100
Subject: [PATCH] full check style before unittest

---
 .github/workflows/python_actions.yml | 38 +++++++++++++---------------
 1 file changed, 17 insertions(+), 21 deletions(-)

diff --git a/.github/workflows/python_actions.yml b/.github/workflows/python_actions.yml
index 5fce6acbf..8f66efc7a 100644
--- a/.github/workflows/python_actions.yml
+++ b/.github/workflows/python_actions.yml
@@ -25,7 +25,7 @@ jobs:
       matrix:
         include:
           - runner: ubuntu-24.04
-            python-version: "3.12"
+            python-version: "3.13"
             checks: 'tests'
 
           - runner: ubuntu-22.04
@@ -33,14 +33,6 @@ jobs:
             checks: 'full'
             coverage: 'coverage'
 
-          - runner: ubuntu-20.04
-            python-version: 3.8
-            checks: 'full'
-
-          - runner: ubuntu-latest
-            python-version: "3.13"
-            checks: 'tests'
-
           - runner: ubuntu-latest
             python-version: "3.11"
             checks: 'tests'
@@ -53,6 +45,10 @@ jobs:
             python-version: 3.9
             checks: 'tests'
 
+          - runner: ubuntu-20.04
+            python-version: 3.8
+            checks: 'full'
+
           - runner: windows-latest
             python-version: "3.12"
             checks: 'tests'
@@ -106,18 +102,6 @@ jobs:
     - name: Run Install
       uses: ./support/actions/run-install
 
-    - name: Test with pytest (and coverage)
-      if: ${{ matrix.checks == 'full' }}
-      uses: ./support/actions/pytest
-      env:
-        SPALLOC_USER: ${{ secrets.SPALLOC_USER }}
-        SPALLOC_PASSWORD: ${{ secrets.SPALLOC_PASSWORD }}
-      with:
-        tests: unittests
-        coverage: ${{ matrix.coverage == 'coverage' }}
-        cover-packages: spinnman, spinnman_integration_tests
-        coveralls-token: ${{ secrets.GITHUB_TOKEN }}
-
     - name: Simple pytest
       if: ${{ matrix.checks == 'tests'}}
       env:
@@ -144,6 +128,18 @@ jobs:
       if: ${{ matrix.checks == 'full' }}
       run: mypy spinnman
 
+    - name: Test with pytest (and coverage)
+      if: ${{ matrix.checks == 'full' }}
+      uses: ./support/actions/pytest
+      env:
+        SPALLOC_USER: ${{ secrets.SPALLOC_USER }}
+        SPALLOC_PASSWORD: ${{ secrets.SPALLOC_PASSWORD }}
+      with:
+        tests: unittests
+        coverage: ${{ matrix.coverage == 'coverage' }}
+        cover-packages: spinnman, spinnman_integration_tests
+        coveralls-token: ${{ secrets.GITHUB_TOKEN }}
+
     - name: Run rat copyright enforcement
       if: ${{ matrix.checks == 'validate' }}
       uses: ./support/actions/check-copyrights