Skip to content

Commit

Permalink
Rollup merge of rust-lang#61096 - ehuss:tidy-license-short-circuit, r…
Browse files Browse the repository at this point in the history
…=Centril

tidy: don't short-circuit on license error

If there is more than one license error, tidy would only print the first
error. This changes it so that all license errors are printed.
  • Loading branch information
Centril authored May 25, 2019
2 parents 2ac8ec0 + 46ffb6a commit 835281f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/tools/tidy/src/deps.rs
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ pub fn check(path: &Path, bad: &mut bool) {
}

let toml = dir.path().join("Cargo.toml");
*bad = *bad || !check_license(&toml);
*bad = !check_license(&toml) || *bad;
}
assert!(saw_dir, "no vendored source");
}
Expand Down

0 comments on commit 835281f

Please sign in to comment.