Update Rust edition to 2024 (#153)

**Stack**:
- #156
- #155
- #154
- #153 ⬅


⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do
not merge manually using the UI - doing so may have unexpected results.*

Author: ksew1

Cargo.toml

@@ -8,7 +8,7 @@ members = [
 
 [workspace.package]
 version = "0.4.0"
-edition = "2021"
+edition = "2024"
 
 [workspace.dependencies]
 anyhow = "1.0.95"

crates/cairo-coverage-core/benches/benchmarks/starknet_staking.rs

@@ -1,5 +1,5 @@
 use crate::benchmarks::{config, run_options, trace_files_for_benches};
-use criterion::{criterion_group, Criterion};
+use criterion::{Criterion, criterion_group};
 use std::hint::black_box;
 
 const PROJECT_NAME: &str = "starknet-staking";

crates/cairo-coverage-core/src/build/coverage_input.rs

@@ -7,7 +7,7 @@ use crate::loading::execution_data::ExecutionData;
 use anyhow::Result;
 use cairo_lang_sierra::program::Program;
 use cairo_lang_sierra_to_casm::compiler::{CairoProgramDebugInfo, SierraToCasmConfig};
-use cairo_lang_sierra_to_casm::metadata::{calc_metadata, MetadataComputationConfig};
+use cairo_lang_sierra_to_casm::metadata::{MetadataComputationConfig, calc_metadata};
 
 /// All necessary data for the coverage analysis.
 #[derive(Clone)]

crates/cairo-coverage-core/src/build/executed_statement_count.rs

@@ -52,7 +52,7 @@ fn points_to_different_statement(
     last_idx: Option<&StatementIdx>,
     current_idx: StatementIdx,
 ) -> bool {
-    last_idx.map_or(true, |last_idx| {
+    last_idx.is_none_or(|last_idx| {
         statement_information_map.get(last_idx) != statement_information_map.get(&current_idx)
     })
 }

crates/cairo-coverage-core/src/build/filter/ignore_matcher.rs

@@ -1,8 +1,8 @@
 use crate::build::filter::statement_category_filter::VIRTUAL_FILE_REGEX;
 use anyhow::{Error, Result};
 use camino::{Utf8Path, Utf8PathBuf};
-use ignore::gitignore::Gitignore;
 use ignore::Match;
+use ignore::gitignore::Gitignore;
 
 const CAIRO_COVERAGE_IGNORE: &str = ".cairo-coverage-ignore";
 

crates/cairo-coverage-core/src/hashmap_utils/merge.rs

@@ -1,5 +1,5 @@
-use std::collections::hash_map::Entry;
 use std::collections::HashMap;
+use std::collections::hash_map::Entry;
 use std::hash::Hash;
 
 /// Trait for merging two values

crates/cairo-coverage/src/args/run.rs

@@ -1,4 +1,4 @@
-use anyhow::{ensure, Result};
+use anyhow::{Result, ensure};
 use camino::Utf8PathBuf;
 use clap::{Parser, ValueEnum};
 

crates/cairo-coverage/src/commands/clean.rs

@@ -41,8 +41,8 @@ pub fn run(
 mod tests {
     use super::super::clean;
     use super::CleanArgs;
-    use assert_fs::fixture::PathChild;
     use assert_fs::TempDir;
+    use assert_fs::fixture::PathChild;
     use cairo_coverage_test_utils::{CreateFile, Utf8PathBufConversion};
 
     #[test]

crates/cairo-coverage/tests/e2e/general.rs

@@ -18,12 +18,14 @@ fn simple_with_tests() {
 
 #[test]
 fn simple_with_output_path() {
-    assert!(TestProject::new("simple")
-        .coverage_args(&["--output-path", "custom_output.lcov"])
-        .run()
-        .dir()
-        .child("custom_output.lcov")
-        .exists());
+    assert!(
+        TestProject::new("simple")
+            .coverage_args(&["--output-path", "custom_output.lcov"])
+            .run()
+            .dir()
+            .child("custom_output.lcov")
+            .exists()
+    );
 }
 
 #[test]

crates/cairo-coverage/tests/helpers/test_project.rs

@@ -1,8 +1,8 @@
-use assert_fs::fixture::PathCopy;
 use assert_fs::TempDir;
+use assert_fs::fixture::PathCopy;
 use cairo_coverage_test_utils::read_files_from_dir;
 use camino::Utf8PathBuf;
-use snapbox::cmd::{cargo_bin, Command as SnapboxCommand};
+use snapbox::cmd::{Command as SnapboxCommand, cargo_bin};
 use std::fs;
 use std::path::PathBuf;
 use which::which;