summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthias Beyer <matthias.beyer@ifm.com>2022-08-12 09:26:08 +0200
committerMatthias Beyer <matthias.beyer@ifm.com>2022-08-12 10:33:01 +0200
commita3ce539c441f9ed9c5ce282b3b933487aeadca8b (patch)
tree702985af20a314c3cb46c355808d162fa6ae8ffc
parent4e10ac79a1910c0d1762a49f75f3dfb5e705c831 (diff)
Do not depend on fmt/clippy in long-running jobs
The build and test jobs take a really long time (approx 17 minutes for the arm32 test job). We cannot afford to wait on cargo-fmt and cargo-clippy with those, so remove the dependencies here to get a faster overall time. Instead, only depend on the "changes" job, so we only run if there are rust/workflow changes. Signed-off-by: Matthias Beyer <matthias.beyer@ifm.com>
-rw-r--r--.github/workflows/pull-request-checks.yml12
1 files changed, 8 insertions, 4 deletions
diff --git a/.github/workflows/pull-request-checks.yml b/.github/workflows/pull-request-checks.yml
index 0bad7e84..e4784cab 100644
--- a/.github/workflows/pull-request-checks.yml
+++ b/.github/workflows/pull-request-checks.yml
@@ -184,7 +184,8 @@ jobs:
runs-on: Ubuntu-20.04
env:
RUSTFLAGS: -D warnings
- needs: [cargo-fmt, cargo-clippy]
+ needs: changes
+ if: ${{ needs.changes.outputs.rust == 'true' || needs.changes.outputs.workflows == 'true' }}
steps:
- name: Checkout
@@ -225,7 +226,8 @@ jobs:
runs-on: Ubuntu-20.04
env:
RUSTFLAGS: -D warnings
- needs: [cargo-fmt, cargo-clippy]
+ needs: changes
+ if: ${{ needs.changes.outputs.rust == 'true' || needs.changes.outputs.workflows == 'true' }}
steps:
- name: Checkout
@@ -258,7 +260,8 @@ jobs:
runs-on: Ubuntu-20.04
env:
RUSTFLAGS: -D warnings
- needs: [cargo-fmt, cargo-clippy]
+ needs: changes
+ if: ${{ needs.changes.outputs.rust == 'true' || needs.changes.outputs.workflows == 'true' }}
steps:
- name: checkout
@@ -286,7 +289,8 @@ jobs:
runs-on: Ubuntu-20.04
env:
RUSTFLAGS: -D warnings
- needs: [cargo-fmt, cargo-clippy]
+ needs: changes
+ if: ${{ needs.changes.outputs.rust == 'true' || needs.changes.outputs.workflows == 'true' }}
steps:
- name: checkout