diff options
author | Matthias Beyer <matthias.beyer@ifm.com> | 2022-08-12 09:26:08 +0200 |
---|---|---|
committer | Matthias Beyer <matthias.beyer@ifm.com> | 2022-08-12 10:33:01 +0200 |
commit | a3ce539c441f9ed9c5ce282b3b933487aeadca8b (patch) | |
tree | 702985af20a314c3cb46c355808d162fa6ae8ffc | |
parent | 4e10ac79a1910c0d1762a49f75f3dfb5e705c831 (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.yml | 12 |
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 |