From b142b09e22721da5f68fe76893d50d734225df2b Mon Sep 17 00:00:00 2001
From: Joe Rickerby <joerick@mac.com>
Date: Fri, 26 Mar 2021 14:33:55 +0000
Subject: [PATCH] ci: make GHA pre-commit check a precursor to main test (#624)

* Remove GHA pre-commit check

This check was the same as the check performed by pre-commit.ci, except that the pre-commit.ci is a bit faster and can fix some issues itself, so I'm removing the GHA one in favour of the pre-commit one.

* Add back precommit check and use it as a precursor

This will prevent us wasting CI time on builds that failed precommit
---
 .github/workflows/test.yml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 2ae8ce95..0eddf360 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -21,6 +21,7 @@ jobs:
 
   test:
     name: Test cibuildwheel on ${{ matrix.os }}
+    needs: pre-commit
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
@@ -70,6 +71,7 @@ jobs:
 
   test-emulated:
     name: Test emulated cibuildwheel using qemu
+    needs: pre-commit
     runs-on: ubuntu-latest
     timeout-minutes: 180
     steps: