From fb9e181a1a777bc2f42bd7c172be1ebddb3b68fa Mon Sep 17 00:00:00 2001 From: Daniel McCarney Date: Mon, 20 Mar 2023 10:36:13 -0400 Subject: [PATCH] CI: add fmt and clippy coverage for fuzz workspace. The `fuzz` subdirectory is set up as a separate workspace from the main workspace that contains `rustls` and `examples`. Because of this running `cargo fmt --all` and other similar tooling doesn't include the `fuzz` directory. This can lead to formatting/clippy drift over time. In this commit the `build.yml` config is extended to also run `clippy` and `fmt` on the `fuzz` subdirectory using `--manifest-path`. --- .github/workflows/build.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 66c710bc..46055a42 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -217,6 +217,8 @@ jobs: components: rustfmt - name: Check formatting run: cargo fmt --all -- --check + - name: Check formatting (fuzz workspace) + run: cargo fmt --all --manifest-path=fuzz/Cargo.toml -- --check clippy: name: Clippy @@ -231,6 +233,7 @@ jobs: with: components: clippy - run: cargo clippy --package rustls --all-features -- --deny warnings + - run: cargo clippy --manifest-path=fuzz/Cargo.toml --all-features -- --deny warnings clippy-nightly: name: Clippy (Nightly) @@ -245,3 +248,4 @@ jobs: with: components: clippy - run: cargo clippy --package rustls --all-features + - run: cargo clippy --manifest-path=fuzz/Cargo.toml --all-features -- --deny warnings