Skip to content

Commit 6868932

Browse files
committed
fix clippy: needless_for_each
1 parent 9673e8c commit 6868932

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/main.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -138,17 +138,17 @@ fn filter_files(changed_files: &Vec<String>, include_patterns_filters: &HashSet<
138138
let mut hash_set_filtered_files = HashSet::new();
139139

140140
for changed_file in changed_files {
141-
include_patterns_filters.iter().for_each(|pattern| {
141+
for pattern in include_patterns_filters {
142142
if Pattern::new(pattern).expect("Failed to create pattern").matches(changed_file) {
143143
hash_set_filtered_files.insert(changed_file.to_string());
144144
}
145145

146-
exclude_patterns_filters.iter().for_each(|pattern| {
146+
for pattern in exclude_patterns_filters {
147147
if Pattern::new(pattern).expect("Failed to create pattern").matches(changed_file) {
148148
hash_set_filtered_files.remove(changed_file);
149149
}
150-
});
151-
});
150+
}
151+
}
152152
}
153153

154154
hash_set_filtered_files
@@ -162,13 +162,13 @@ fn categorize_filters(filters: &Vec<PatternFilter>) -> (HashSet<String>, HashSet
162162
let mut exclude_patterns_filters: HashSet<String> = HashSet::new();
163163
let mut include_patterns_filters: HashSet<String> = HashSet::new();
164164

165-
filters.iter().for_each(|pattern_filter| {
165+
for pattern_filter in filters {
166166
if pattern_filter.exclude {
167167
exclude_patterns_filters.insert(pattern_filter.clone().pattern);
168168
} else {
169169
include_patterns_filters.insert(pattern_filter.clone().pattern);
170170
}
171-
});
171+
}
172172

173173
(include_patterns_filters, exclude_patterns_filters)
174174
}

0 commit comments

Comments
 (0)