From c70dce0dde36efcf07f2d50d2711676c46fe1b76 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 10:54:26 -0400
Subject: [PATCH 01/19] ci: attempt windows build fix

---
 .github/workflows/python-package.yml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 21467eeb..9eefb328 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -88,8 +88,8 @@ jobs:
           pytest -v --benchmark-disable -n auto
 
   build:
-    needs: [test]
-    if: "startsWith(github.ref, 'refs/tags/')"
+    # needs: [test]
+    # if: "startsWith(github.ref, 'refs/tags/')"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true
@@ -104,7 +104,7 @@ jobs:
         uses: docker/setup-qemu-action@v3
         with:
           platforms: arm64
-      - uses: pypa/cibuildwheel@v2.17.0
+      - uses: pypa/cibuildwheel@v2.20.0
         env:
           CIBW_SKIP: cp36-* pp*-win* pp*-macosx* *_i686
           CIBW_TEST_SKIP: "*-win_arm64"

From f99de1f3820636c8313579d3680c96f74bcd334c Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 12:02:03 -0400
Subject: [PATCH 02/19] ci: re-enabled build condition

---
 .github/workflows/python-package.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 9eefb328..a9148005 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -88,8 +88,8 @@ jobs:
           pytest -v --benchmark-disable -n auto
 
   build:
-    # needs: [test]
-    # if: "startsWith(github.ref, 'refs/tags/')"
+    needs: [test]
+    if: "startsWith(github.ref, 'refs/tags/')"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From 10ce29351af1752885898a8f668fe658893e2c54 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 12:05:53 -0400
Subject: [PATCH 03/19] ci: test conditional build with [ci build]

---
 .github/workflows/python-package.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index a9148005..7178fe96 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -89,7 +89,7 @@ jobs:
 
   build:
     needs: [test]
-    if: "startsWith(github.ref, 'refs/tags/')"
+    if: ${{ startsWith(github.ref, 'refs/tags/') or contains(github.event.head_commit.message, '[ci build]') }}
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true
@@ -121,7 +121,7 @@ jobs:
   release:
     name: Release
     runs-on: ubuntu-latest
-    if: "startsWith(github.ref, 'refs/tags/')"
+    if: ${{ startsWith(github.ref, 'refs/tags/') }}
     needs: [build]
     steps:
       - uses: actions/download-artifact@v4

From 9895a473dafd7e85aeec8265faff98abdb0a7b24 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 12:07:05 -0400
Subject: [PATCH 04/19] ci: syntax fix

[ci build]
---
 .github/workflows/python-package.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 7178fe96..cdb55140 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -89,7 +89,7 @@ jobs:
 
   build:
     needs: [test]
-    if: ${{ startsWith(github.ref, 'refs/tags/') or contains(github.event.head_commit.message, '[ci build]') }}
+    if: ${{ startsWith(github.ref, 'refs/tags/') || contains(github.event.head_commit.message, '[ci build]') }}
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From ca9620e6d7d0bc3a41a9182055044fd0e21cdd6a Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 12:17:30 -0400
Subject: [PATCH 05/19] ci: force build output

[ci build]
---
 .github/workflows/python-package.yml | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index cdb55140..cf79e1f1 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -49,6 +49,13 @@ jobs:
         with:
           name: wheels
           path: dist
+      - name: Check for [ci build] in commit message
+        run: |
+          if git log -1 --pretty=format:"%B" | grep "\[ci build\]" ; then
+            echo "force-build=true" >> "$GITHUB_OUTPUT"
+          else
+            echo "force-build=false" >> "$GITHUB_OUTPUT"
+          fi
 
   test:
     needs: [lint]
@@ -63,7 +70,6 @@ jobs:
           # https://github.com/actions/setup-python/issues/852
           - os: macos-latest
             python-version: "3.7"
-
     steps:
       - uses: actions/checkout@v4
         with:
@@ -87,9 +93,12 @@ jobs:
         run: |
           pytest -v --benchmark-disable -n auto
 
+  # NOTE: build step only runs on tag builds or when the commit message contains
+  # "[ci build]" for testing
   build:
-    needs: [test]
-    if: ${{ startsWith(github.ref, 'refs/tags/') || contains(github.event.head_commit.message, '[ci build]') }}
+    # needs: [lint, test]
+    needs: [lint]
+    if: "startsWith(github.ref, 'refs/tags/') || needs.lint.outputs.force-build == 'true'"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true
@@ -118,6 +127,7 @@ jobs:
           name: wheels-${{ runner.os }}
           path: dist
 
+  # NOTE: release step only runs on tag builds
   release:
     name: Release
     runs-on: ubuntu-latest

From 7c3d51e6433cd8e4fdbe117c443407a3e462e7d5 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 12:52:29 -0400
Subject: [PATCH 06/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index cf79e1f1..2f708d20 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -93,12 +93,18 @@ jobs:
         run: |
           pytest -v --benchmark-disable -n auto
 
+  foo:
+    needs: [lint]
+    runs-on: ubuntu-latest
+    steps:
+      - run: echo "${{ needs.lint.outputs.force-build }}"
+
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
     # needs: [lint, test]
     needs: [lint]
-    if: "startsWith(github.ref, 'refs/tags/') || needs.lint.outputs.force-build == 'true'"
+    if: "${{ startsWith(github.ref, 'refs/tags/') || needs.lint.outputs.force-build == 'true' }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From 8ccde12a75add181914f9c98f4d83cc9ee1a970f Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:06:13 -0400
Subject: [PATCH 07/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 2f708d20..f258fef5 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -49,13 +49,13 @@ jobs:
         with:
           name: wheels
           path: dist
-      - name: Check for [ci build] in commit message
+      - name: Output commit message
         run: |
-          if git log -1 --pretty=format:"%B" | grep "\[ci build\]" ; then
-            echo "force-build=true" >> "$GITHUB_OUTPUT"
-          else
-            echo "force-build=false" >> "$GITHUB_OUTPUT"
-          fi
+          {
+            echo 'commit-message<<CI_EOF'
+            git show -s --format="%B"
+            echo CI_EOF
+          } >> "$GITHUB_OUTPUT"
 
   test:
     needs: [lint]
@@ -97,14 +97,14 @@ jobs:
     needs: [lint]
     runs-on: ubuntu-latest
     steps:
-      - run: echo "${{ needs.lint.outputs.force-build }}"
+      - run: echo "${{ needs.lint.outputs.commit-message }}"
 
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
     # needs: [lint, test]
     needs: [lint]
-    if: "${{ startsWith(github.ref, 'refs/tags/') || needs.lint.outputs.force-build == 'true' }}"
+    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.lint.outputs.commit-message, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From ff027458fbf8c56178b92dd177095be3f3c29d2f Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:09:23 -0400
Subject: [PATCH 08/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index f258fef5..995c8818 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -52,7 +52,7 @@ jobs:
       - name: Output commit message
         run: |
           {
-            echo 'commit-message<<CI_EOF'
+            echo 'commit<<CI_EOF'
             git show -s --format="%B"
             echo CI_EOF
           } >> "$GITHUB_OUTPUT"
@@ -94,17 +94,22 @@ jobs:
           pytest -v --benchmark-disable -n auto
 
   foo:
-    needs: [lint]
     runs-on: ubuntu-latest
     steps:
-      - run: echo "${{ needs.lint.outputs.commit-message }}"
+      - name: Output commit message
+        run: |
+          {
+            echo 'commit<<CI_EOF'
+            git show -s --format="%B"
+            echo CI_EOF
+          } >> "$GITHUB_OUTPUT"
 
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
     # needs: [lint, test]
-    needs: [lint]
-    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.lint.outputs.commit-message, '[ci build]') }}"
+    needs: [foo]
+    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.foo.outputs.commit, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From a2768f86c21f6bba6a6e6120b3b25abc7931aa16 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:10:05 -0400
Subject: [PATCH 09/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 995c8818..1da24462 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -96,6 +96,7 @@ jobs:
   foo:
     runs-on: ubuntu-latest
     steps:
+      - uses: actions/checkout@v4
       - name: Output commit message
         run: |
           {

From ff5755875a3326ee00347e4e0a9f10374fb000af Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:12:12 -0400
Subject: [PATCH 10/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 1da24462..c3ca939a 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -99,12 +99,20 @@ jobs:
       - uses: actions/checkout@v4
       - name: Output commit message
         run: |
+          git show -s --format="%B"
           {
             echo 'commit<<CI_EOF'
             git show -s --format="%B"
             echo CI_EOF
           } >> "$GITHUB_OUTPUT"
 
+  bar:
+    runs-on: ubuntu-latest
+    needs: [foo]
+    steps:
+      - name: Get commit message
+        run: echo "${{ needs.foo.outputs.commit }}"
+
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:

From 3df24e78a75e1f5c070d06da7162399f099ec928 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:14:14 -0400
Subject: [PATCH 11/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index c3ca939a..8dd97fad 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -28,6 +28,7 @@ jobs:
       - uses: actions/checkout@v4
         with:
           submodules: true
+          ref: ${{ github.event.pull_request.head.sha }} # gets correct commit message
       - uses: actions/setup-python@v5
         with:
           python-version: "3.10"
@@ -97,6 +98,8 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha }} # gets correct commit message
       - name: Output commit message
         run: |
           git show -s --format="%B"

From a5d715eb36b4f6f94f9cfe235a04f8bd81bc3fdb Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:17:08 -0400
Subject: [PATCH 12/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 8dd97fad..b691d1f5 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -96,15 +96,18 @@ jobs:
 
   foo:
     runs-on: ubuntu-latest
+    outputs:
+      commit-message: ${{ steps.get-commit-message.outputs.commit-message }}
     steps:
       - uses: actions/checkout@v4
         with:
           ref: ${{ github.event.pull_request.head.sha }} # gets correct commit message
       - name: Output commit message
+        id: get-commit-message
         run: |
           git show -s --format="%B"
           {
-            echo 'commit<<CI_EOF'
+            echo 'commit-message<<CI_EOF'
             git show -s --format="%B"
             echo CI_EOF
           } >> "$GITHUB_OUTPUT"
@@ -114,7 +117,7 @@ jobs:
     needs: [foo]
     steps:
       - name: Get commit message
-        run: echo "${{ needs.foo.outputs.commit }}"
+        run: echo "${{ needs.foo.outputs.commit-message }}"
 
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing

From c8615a140e40fb12cca2572307d06bfa7438c8cc Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:19:47 -0400
Subject: [PATCH 13/19] ci: hopefully final [ci build] test

---
 .github/workflows/python-package.yml | 33 +++++-----------------------
 1 file changed, 5 insertions(+), 28 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index b691d1f5..b7600231 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -24,6 +24,8 @@ concurrency:
 jobs:
   lint:
     runs-on: ubuntu-latest
+    outputs:
+      commit-message: ${{ steps.get-commit-message.outputs.commit-message }}
     steps:
       - uses: actions/checkout@v4
         with:
@@ -53,7 +55,7 @@ jobs:
       - name: Output commit message
         run: |
           {
-            echo 'commit<<CI_EOF'
+            echo 'commit-message<<CI_EOF'
             git show -s --format="%B"
             echo CI_EOF
           } >> "$GITHUB_OUTPUT"
@@ -94,37 +96,12 @@ jobs:
         run: |
           pytest -v --benchmark-disable -n auto
 
-  foo:
-    runs-on: ubuntu-latest
-    outputs:
-      commit-message: ${{ steps.get-commit-message.outputs.commit-message }}
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          ref: ${{ github.event.pull_request.head.sha }} # gets correct commit message
-      - name: Output commit message
-        id: get-commit-message
-        run: |
-          git show -s --format="%B"
-          {
-            echo 'commit-message<<CI_EOF'
-            git show -s --format="%B"
-            echo CI_EOF
-          } >> "$GITHUB_OUTPUT"
-
-  bar:
-    runs-on: ubuntu-latest
-    needs: [foo]
-    steps:
-      - name: Get commit message
-        run: echo "${{ needs.foo.outputs.commit-message }}"
-
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
     # needs: [lint, test]
-    needs: [foo]
-    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.foo.outputs.commit, '[ci build]') }}"
+    needs: [lint]
+    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.lint.outputs.commit-message, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From 0431e1be3e2ede9725d2a6404f69ced88b698a24 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:26:31 -0400
Subject: [PATCH 14/19] ci: another [ci build] test

---
 .github/workflows/python-package.yml | 29 ++++++++++++++++++++++++++--
 1 file changed, 27 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index b7600231..ab052495 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -96,12 +96,37 @@ jobs:
         run: |
           pytest -v --benchmark-disable -n auto
 
+  foo:
+    runs-on: ubuntu-latest
+    outputs:
+      commit-message: ${{ steps.get-commit-message.outputs.commit-message }}
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha }} # gets correct commit message
+      - name: Output commit message
+        id: get-commit-message
+        run: |
+          git show -s --format="%B"
+          {
+            echo 'commit-message<<CI_EOF'
+            git show -s --format="%B"
+            echo CI_EOF
+          } >> "$GITHUB_OUTPUT"
+
+  bar:
+    runs-on: ubuntu-latest
+    needs: [foo]
+    steps:
+      - name: Get commit message
+        run: echo "${{ needs.foo.outputs.commit-message }}"
+
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
     # needs: [lint, test]
-    needs: [lint]
-    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.lint.outputs.commit-message, '[ci build]') }}"
+    needs: [foo]
+    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.foo.outputs.commit-message, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From 25827ce062960972a3f8a8c1cb69351e3f55a4a8 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:28:14 -0400
Subject: [PATCH 15/19] ci: hopefully final [ci build] test

---
 .github/workflows/python-package.yml | 30 +++-------------------------
 1 file changed, 3 insertions(+), 27 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index ab052495..b27beea3 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -53,6 +53,7 @@ jobs:
           name: wheels
           path: dist
       - name: Output commit message
+        id: get-commit-message
         run: |
           {
             echo 'commit-message<<CI_EOF'
@@ -96,37 +97,12 @@ jobs:
         run: |
           pytest -v --benchmark-disable -n auto
 
-  foo:
-    runs-on: ubuntu-latest
-    outputs:
-      commit-message: ${{ steps.get-commit-message.outputs.commit-message }}
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          ref: ${{ github.event.pull_request.head.sha }} # gets correct commit message
-      - name: Output commit message
-        id: get-commit-message
-        run: |
-          git show -s --format="%B"
-          {
-            echo 'commit-message<<CI_EOF'
-            git show -s --format="%B"
-            echo CI_EOF
-          } >> "$GITHUB_OUTPUT"
-
-  bar:
-    runs-on: ubuntu-latest
-    needs: [foo]
-    steps:
-      - name: Get commit message
-        run: echo "${{ needs.foo.outputs.commit-message }}"
-
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
     # needs: [lint, test]
-    needs: [foo]
-    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.foo.outputs.commit-message, '[ci build]') }}"
+    needs: [lint]
+    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.lint.outputs.commit-message, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From f5be944b9a58c2dbebeeb0859b18b2ca8153f281 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:30:42 -0400
Subject: [PATCH 16/19] ci: revert to default settings

---
 .github/workflows/python-package.yml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index b27beea3..8a028b67 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -100,8 +100,7 @@ jobs:
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
-    # needs: [lint, test]
-    needs: [lint]
+    needs: [lint, test]
     if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.lint.outputs.commit-message, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:

From 743b5f24882d2e0a2fa7383395136aa8a7f81411 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:41:05 -0400
Subject: [PATCH 17/19] ci: more reliable commit message with meta step

[ci build]
---
 .github/workflows/python-package.yml | 32 ++++++++++++++++++----------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 8a028b67..4f9f4966 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -22,10 +22,27 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  lint:
+  meta:
     runs-on: ubuntu-latest
     outputs:
       commit-message: ${{ steps.get-commit-message.outputs.commit-message }}
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha || github.sha }} # gets correct commit message
+      - name: Show commit
+        run: git show -s
+      - name: Output commit message
+        id: get-commit-message
+        run: |
+          {
+            echo 'commit-message<<CI_EOF'
+            git show -s --format="%B"
+            echo CI_EOF
+          } >> "$GITHUB_OUTPUT"
+
+  lint:
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
         with:
@@ -52,14 +69,6 @@ jobs:
         with:
           name: wheels
           path: dist
-      - name: Output commit message
-        id: get-commit-message
-        run: |
-          {
-            echo 'commit-message<<CI_EOF'
-            git show -s --format="%B"
-            echo CI_EOF
-          } >> "$GITHUB_OUTPUT"
 
   test:
     needs: [lint]
@@ -100,8 +109,9 @@ jobs:
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
-    needs: [lint, test]
-    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.lint.outputs.commit-message, '[ci build]') }}"
+    # needs: [meta, test]
+    needs: [meta]
+    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.meta.outputs.commit-message, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true

From 991f03af1dfe7b68c7f1a4bfd735dead533c1eee Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 13:42:34 -0400
Subject: [PATCH 18/19] ci: revert to default settings

---
 .github/workflows/python-package.yml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 4f9f4966..6b40b557 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -109,8 +109,7 @@ jobs:
   # NOTE: build step only runs on tag builds or when the commit message contains
   # "[ci build]" for testing
   build:
-    # needs: [meta, test]
-    needs: [meta]
+    needs: [meta, test]
     if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.meta.outputs.commit-message, '[ci build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:

From 2e1a94d0a252d2efe56d8e55f7060e7a4bbb7239 Mon Sep 17 00:00:00 2001
From: Nick Frasser <1693461+nfrasser@users.noreply.github.com>
Date: Tue, 10 Sep 2024 14:01:22 -0400
Subject: [PATCH 19/19] ci: build -> test build

---
 .github/workflows/python-package.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 6b40b557..f2974b87 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -107,10 +107,10 @@ jobs:
           pytest -v --benchmark-disable -n auto
 
   # NOTE: build step only runs on tag builds or when the commit message contains
-  # "[ci build]" for testing
+  # "[ci test build]" for testing
   build:
     needs: [meta, test]
-    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.meta.outputs.commit-message, '[ci build]') }}"
+    if: "${{ startsWith(github.ref, 'refs/tags/') || contains(needs.meta.outputs.commit-message, '[ci test build]') }}"
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: true