Run compatibility checks only in main branch.

Moved the compatibility check into it's own workflow. The workflow uses a flag to specify whether the step is enabled, as it cannot just be skipped, due to the all-jobs-complete dependency on the compatibility-test.

Author: jeffsmale90

.github/workflows/build-lint-test.yml

@@ -83,29 +83,3 @@ jobs:
             echo "Working tree dirty at end of job"
             exit 1
           fi
-
-  compatibility-test:
-    name: Compatibility test
-    needs: prepare
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        node-version: [18.x, 20.x, 22.x]
-    steps:
-      - name: Checkout and setup environment
-        uses: MetaMask/action-checkout-and-setup@v1
-        with:
-          is-high-risk-environment: false
-          node-version: ${{ matrix.node-version }}
-      - name: Install dependencies via Yarn
-        run: rm yarn.lock && YARN_ENABLE_IMMUTABLE_INSTALLS=false yarn
-      - run: yarn test
-      - name: Restore lockfile
-        run: git restore yarn.lock
-      - name: Require clean working directory
-        shell: bash
-        run: |
-          if ! git diff --exit-code; then
-            echo "Working tree dirty at end of job"
-            exit 1
-          fi

.github/workflows/compatibility-test.yml

@@ -0,0 +1,39 @@
+name: Compatibility Test
+
+on:
+  workflow_call:
+    inputs:
+      enabled:
+        required: false
+        type: boolean
+        default: true
+
+jobs:
+  enabled-check:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Skipping compatibility test
+        if: inputs.enabled != true
+        run: echo "Skipping compatibility test"
+
+  compatibility-test:
+    needs: enabled-check
+    if: inputs.enabled == true
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node-version: [18.x, 20.x, 22.x]
+    steps:
+      - uses: MetaMask/action-checkout-and-setup@v1
+        with:
+          is-high-risk-environment: false
+          node-version: ${{ matrix.node-version }}
+      - run: rm yarn.lock && YARN_ENABLE_IMMUTABLE_INSTALLS=false yarn
+      - run: yarn test
+      - run: git restore yarn.lock
+      - shell: bash
+        run: |
+          if ! git diff --exit-code; then
+            echo "Working tree dirty at end of job"
+            exit 1
+          fi

.github/workflows/main.yml

@@ -38,13 +38,20 @@ jobs:
     name: Build, lint, and test
     uses: ./.github/workflows/build-lint-test.yml
 
+  compatibility-test:
+    name: Compatibility test
+    uses: ./.github/workflows/compatibility-test.yml
+    with:
+      enabled: ${{ github.ref == 'refs/heads/main' }}
+
   all-jobs-completed:
     name: All jobs completed
     runs-on: ubuntu-latest
     needs:
       - check-workflows
       - analyse-code
       - build-lint-test
+      - compatibility-test
     outputs:
       PASSED: ${{ steps.set-output.outputs.PASSED }}
     steps: