style: use explicit loop instead of .for_each() in simple cases

samueltardieu · Oct 2, 2024 · f0c8f9f · f0c8f9f
1 parent 49d7a6f
commit f0c8f9f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -136,6 +136,7 @@ testutils = { path = "lib/testutils" }
 explicit_iter_loop = "deny"
 flat_map_option = "deny"
 implicit_clone = "deny"
+needless_for_each = "deny"
 redundant_else = "deny"
 semicolon_if_nothing_returned = "deny"
 uninlined_format_args = "deny"

diff --git a/cli/src/movement_util.rs b/cli/src/movement_util.rs
@@ -101,7 +101,7 @@ impl Direction {
 
         let template = workspace_command.commit_summary_template();
         let mut cmd_err = user_error(err_msg);
-        commits.iter().for_each(|commit| {
+        for commit in commits {
             cmd_err.add_formatted_hint_with(|formatter| {
                 if args.should_edit {
                     write!(formatter, "Working copy: ")?;
@@ -110,7 +110,7 @@ impl Direction {
                 }
                 template.format(commit, formatter)
             });
-        });
+        }
 
         cmd_err
     }