Skip to content

Commit

Permalink
ci: make sure to not update lockfile as part of style checks (near#10104
Browse files Browse the repository at this point in the history
)

This shouldn’t be accidentally happening especially in cases where user
might have passed `--locked` to the `cargo test` command or something
like that.
  • Loading branch information
nagisa authored Nov 6, 2023
1 parent 5f942be commit 3542b1a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test-utils/style/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ fn clippy() {
let cargo = std::env::var_os("CARGO").unwrap_or(OsString::from("cargo"));
let mut cmd = Command::new(cargo);
cargo_env(&mut cmd);
cmd.args(&["clippy", "--all-targets", "--all-features", "--"]);
cmd.args(&["clippy", "--all-targets", "--all-features", "--locked", "--"]);
cmd.args(&[
"-Aclippy::all",
"-Dwarnings",
Expand All @@ -69,7 +69,7 @@ fn deny() {
let cargo = std::env::var_os("CARGO").unwrap_or(OsString::from("cargo"));
let mut cmd = Command::new(cargo);
cargo_env(&mut cmd);
cmd.args(&["deny", "--all-features", "check", "bans"]);
cmd.args(&["deny", "--all-features", "--locked", "check", "bans"]);
ensure_success(cmd);
}

Expand All @@ -78,6 +78,6 @@ fn themis() {
let cargo = std::env::var_os("CARGO").unwrap_or(OsString::from("cargo"));
let mut cmd = Command::new(cargo);
cargo_env(&mut cmd);
cmd.args(&["run", "-p", "themis"]);
cmd.args(&["run", "--locked", "-p", "themis"]);
ensure_success(cmd);
}

0 comments on commit 3542b1a

Please sign in to comment.