Skip to content

Commit

Permalink
Deny Clippy warning lints in the CI
Browse files Browse the repository at this point in the history
No need to pass --verbose again, since it can trip sccache up and the
info is not that valuable, since it's the last pass in the CI suite.
  • Loading branch information
Xanewok committed Oct 31, 2023
1 parent bf1bba4 commit 1313899
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions crates/infra/cli/src/commands/lint/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -65,11 +65,13 @@ impl OrderedCommand for LintCommand {
}

fn run_clippy() -> Result<()> {
Command::new("cargo")
.flag("clippy")
.flag("--")
.flag("--verbose")
.run()
let mut clippy = Command::new("cargo").flag("clippy").flag("--");

if GitHub::is_running_in_ci() {
clippy = clippy.property("-D", "warnings");
}

clippy.run()
}

fn run_cargo_fmt() -> Result<()> {
Expand Down

0 comments on commit 1313899

Please sign in to comment.