ci: separate build matrix from workflow

Author: castastrophe

.github/workflows/build.yml

@@ -24,7 +24,7 @@ on:
                 default: "20"
             experimental:
                 required: false
-                default: "false"
+                default: false
             ref:
                 description: "The branch or tag to checkout"
                 required: false
@@ -154,6 +154,14 @@ jobs:
                     exit 1
                 fi
 
+            # If there are changes, capture the changes and upload them as an artifact
+            - name: Capture changes
+              if: ${{ failure() }}
+              id: capture-changes
+              run: |
+                git diff
+                git diff > changes.diff
+
             - name: Upload changes
               if: ${{ failure() }}
               uses: actions/upload-artifact@v4

.github/workflows/development.yml

@@ -52,9 +52,13 @@ jobs:
                 system:
                     - macos-latest
                     - ubuntu-latest
-                    # - windows-latest # todo: debug token style-dictionary failures on windows
                 node-version:
                     - 20
+                experimental:
+                    - false
+                include:
+                    - system: windows-latest
+                      experimental: true
         uses: ./.github/workflows/build.yml
         with:
             system: ${{ matrix.system }}
@@ -177,7 +181,7 @@ jobs:
     # -------------------------------------------------------------
     vrt:
         name: Testing
-        if: contains(github.event.pull_request.labels.*.name, 'run_vrt') || ((github.event.pull_request.draft != true || contains(github.event.pull_request.labels.*.name, 'run_ci')) && github.event.pull_request.mergeable == true)
+        if: ${{ contains(github.event.pull_request.labels.*.name, 'run_vrt') || ((github.event.pull_request.draft != true || contains(github.event.pull_request.labels.*.name, 'run_ci')) && github.event.pull_request.mergeable == true) }}
         uses: ./.github/workflows/vrt.yml
         with:
             skip: ${{ github.base_ref == 'spectrum-two' || contains(github.event.pull_request.labels.*.name, 'skip_vrt') }}
@@ -192,7 +196,7 @@ jobs:
         # The build step ensures we are leveraging the cache for the build
         needs: [vrt]
         # Note: the goal here is to allow vrt to be skipped but still require the build to succeed
-        if: ${{ always() }}
+        if: ${{ always() && (needs.vrt.result == 'success' || needs.vrt.result == 'skipped') }}
         uses: ./.github/workflows/publish-site.yml
         with:
             deploy-message: ${{ github.event.pull_request.title }}