From 5fab7ec002d4fed833720ab025cb80ee33fa289a Mon Sep 17 00:00:00 2001 From: Sunjay Varma Date: Mon, 8 Apr 2019 13:51:54 -0400 Subject: [PATCH 1/2] Mention that -D warnings will deny ALL warnings, not just clippy warnings --- README.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/README.md b/README.md index 43b477d2cef..4f573adbc01 100644 --- a/README.md +++ b/README.md @@ -125,6 +125,11 @@ before_script: # etc ``` +Note that adding `-D warnings` will cause your build to fail if **any** warnings are found in your code. +That includes warnings found by rustc (e.g. `dead_code`, etc.). If you want to avoid this and only cause +an error for clippy warnings, use `#![deny(clippy::all)]` in your code or `-D clippy::all` on the command +line. (You can swap `clippy::all` with the specific lint category you are targetting.) + ## Configuration Some lints can be configured in a TOML file named `clippy.toml` or `.clippy.toml`. It contains a basic `variable = value` mapping eg. From a7bfac735390d8c78c9f08e2ba8d87ac997f9dc9 Mon Sep 17 00:00:00 2001 From: Philipp Krones Date: Wed, 10 Apr 2019 22:17:17 -0400 Subject: [PATCH 2/2] Update README.md Co-Authored-By: sunjay --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 4f573adbc01..14952063933 100644 --- a/README.md +++ b/README.md @@ -128,7 +128,7 @@ before_script: Note that adding `-D warnings` will cause your build to fail if **any** warnings are found in your code. That includes warnings found by rustc (e.g. `dead_code`, etc.). If you want to avoid this and only cause an error for clippy warnings, use `#![deny(clippy::all)]` in your code or `-D clippy::all` on the command -line. (You can swap `clippy::all` with the specific lint category you are targetting.) +line. (You can swap `clippy::all` with the specific lint category you are targeting.) ## Configuration