Skip to content

Commit bc4fcec

Browse files
authored
cargo bump-versions properly updates the smoketests lockfile (#4462)
# Description of Changes Previously `bump-versions` would push a failing commit because the smoketests had a diff. This fixes that by having `bump-versions` update that lockfile. # API and ABI breaking changes None. # Expected complexity level and risk 1 # Testing - [x] Ran `cargo bump-versions --all 3.0.0` and confirmed that the lockfile was updated Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
1 parent 350066a commit bc4fcec

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

tools/upgrade-version/src/main.rs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -220,6 +220,13 @@ fn main() -> anyhow::Result<()> {
220220
println!("$> cargo check");
221221
cmd!("cargo", "check").run().expect("Cargo check failed!");
222222

223+
// Update the lockfile in crates/smoketests/modules..
224+
println!("$> cd crates/smoketests/modules && cargo check");
225+
cmd!("cargo", "check")
226+
.dir("crates/smoketests/modules")
227+
.run()
228+
.expect("cargo check in crates/smoketests/modules failed!");
229+
223230
println!("$> pnpm install");
224231
cmd!("pnpm", "install").run().expect("pnpm run build failed!");
225232

0 commit comments

Comments
 (0)