Skip to content

Commit dcd7a63

Browse files
authored
Execute cairo-coverage run instead of cairo-coverage (#3002)
<!-- Reference any GitHub issues resolved by this PR --> Closes # ## Introduced changes <!-- A brief description of the changes --> Running `cairo-coverage` without a subcommand will be deprecated in the next version and will print a warning. To avoid annoying users, this PR ensures Forge is prepared for the upcoming deprecation. It would be nice to include this in `0.38.0`, but it's okay if that doesn't happen. ## Checklist <!-- Make sure all of these are complete --> - [x] Linked relevant issue - [x] Updated relevant documentation - [x] Added relevant tests - [x] Performed self-review of the code - [x] Added changes to `CHANGELOG.md`
1 parent 91e5196 commit dcd7a63

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

crates/forge-runner/src/coverage_api.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ pub fn run_coverage(saved_trace_data_paths: &[PathBuf], coverage_args: &[OsStrin
4545
.collect();
4646

4747
let mut command = Command::new(coverage);
48+
command.arg("run");
4849

4950
if coverage_args.iter().all(|arg| arg != "--output-path") {
5051
let dir_to_save_coverage = PathBuf::from(COVERAGE_DIR);

0 commit comments

Comments
 (0)