From c0d945c0acbd6dd75971f6ee04a8b294471ead7f Mon Sep 17 00:00:00 2001 From: Martin Nordholts Date: Mon, 28 Dec 2020 12:51:56 +0100 Subject: [PATCH] Allow clippy::style lints Turns out GitHub is clever enough to pick up clippy warnings from us just running it, and showing them in PRs under a "Unchanged files with check annotations (beta)" header. The only warnings currently shown are style warnings, and we agreed we don't want to risk putting off contributors over style issues. So explicitly allow (don't warn in the logs for) this category of clippy lints. This means that the only clippy categories left that prints warnings (Warn) are: clippy::complexity clippy::perf And the only category of lints that fails the CI build (Deny) remains to be: clippy::correctness See https://rust-lang.github.io/rust-clippy/master/index.html for a catalog of all lints. --- .github/workflows/CICD.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/CICD.yml b/.github/workflows/CICD.yml index 7a141412..66fb7c76 100644 --- a/.github/workflows/CICD.yml +++ b/.github/workflows/CICD.yml @@ -27,7 +27,7 @@ jobs: uses: actions-rs/cargo@v1 with: command: clippy - args: --all-targets --all-features + args: --all-targets --all-features -- --allow clippy::style - name: Test uses: actions-rs/cargo@v1 with: