From e62f32c16b5058993024f4a29247829fdb76f169 Mon Sep 17 00:00:00 2001 From: Fabian Boehm Date: Sat, 7 Oct 2023 19:32:57 +0200 Subject: [PATCH] CI: Remove --deny=warnings from clippy test This was pretty annoying on rust release day, because it introduced new warnings. Specifically 1.73 introduced a spurious one about PartialOrd and Ord disagreeing when both were in fact #derive-d. --- .github/workflows/rust_checks.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/rust_checks.yml b/.github/workflows/rust_checks.yml index a5661832a..a9f75635d 100644 --- a/.github/workflows/rust_checks.yml +++ b/.github/workflows/rust_checks.yml @@ -29,4 +29,6 @@ jobs: run: | cmake -B build - name: cargo clippy - run: cargo clippy --workspace --all-targets -- --deny=warnings + # This used to have --deny=warnings, but that turns rust release day + # into automatic CI failure day, so we don't do that. + run: cargo clippy --workspace --all-targets