Rename get_filenames to collect_files (#185)

j178 · Feb 17, 2025 · 302217d · 302217d
1 parent facd0af
commit 302217d
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/src/builtin/meta_hooks.rs b/src/builtin/meta_hooks.rs
@@ -8,7 +8,7 @@ use fancy_regex::Regex;
 use itertools::Itertools;
 use rayon::iter::{IntoParallelIterator, ParallelIterator};
 
-use crate::cli::run::{get_filenames, FileFilter, FileOptions};
+use crate::cli::run::{collect_files, CollectOptions, FileFilter};
 use crate::config::Language;
 use crate::hook::{Hook, Project};
 use crate::store::Store;
@@ -21,7 +21,7 @@ pub(crate) async fn check_hooks_apply(
 ) -> Result<(i32, Vec<u8>)> {
     let store = Store::from_settings()?.init()?;
 
-    let input = get_filenames(FileOptions::default().with_all_files(true)).await?;
+    let input = collect_files(CollectOptions::default().with_all_files(true)).await?;
 
     let mut code = 0;
     let mut output = Vec::new();
@@ -89,7 +89,7 @@ pub(crate) async fn check_useless_excludes(
 ) -> Result<(i32, Vec<u8>)> {
     let store = Store::from_settings()?.init()?;
 
-    let input = get_filenames(FileOptions::default().with_all_files(true)).await?;
+    let input = collect_files(CollectOptions::default().with_all_files(true)).await?;
 
     let mut code = 0;
     let mut output = Vec::new();

diff --git a/src/cli/run/filter.rs b/src/cli/run/filter.rs
@@ -164,7 +164,7 @@ impl<'a> FileFilter<'a> {
 }
 
 #[derive(Default)]
-pub struct FileOptions {
+pub struct CollectOptions {
     pub hook_stage: Option<Stage>,
     pub from_ref: Option<String>,
     pub to_ref: Option<String>,
@@ -173,7 +173,7 @@ pub struct FileOptions {
     pub commit_msg_filename: Option<PathBuf>,
 }
 
-impl FileOptions {
+impl CollectOptions {
     pub fn with_all_files(mut self, all_files: bool) -> Self {
         self.all_files = all_files;
         self
@@ -182,8 +182,8 @@ impl FileOptions {
 
 /// Get all filenames to run hooks on.
 #[allow(clippy::too_many_arguments)]
-pub async fn get_filenames(opts: FileOptions) -> Result<Vec<String>> {
-    let FileOptions {
+pub async fn collect_files(opts: CollectOptions) -> Result<Vec<String>> {
+    let CollectOptions {
         hook_stage,
         from_ref,
         to_ref,
@@ -192,7 +192,7 @@ pub async fn get_filenames(opts: FileOptions) -> Result<Vec<String>> {
         commit_msg_filename,
     } = opts;
 
-    let mut filenames = filenames_for_args(
+    let mut filenames = collect_files_from_args(
         hook_stage,
         from_ref,
         to_ref,
@@ -214,7 +214,7 @@ pub async fn get_filenames(opts: FileOptions) -> Result<Vec<String>> {
 }
 
 #[allow(clippy::too_many_arguments)]
-async fn filenames_for_args(
+async fn collect_files_from_args(
     hook_stage: Option<Stage>,
     from_ref: Option<String>,
     to_ref: Option<String>,

diff --git a/src/cli/run/mod.rs b/src/cli/run/mod.rs
@@ -1,4 +1,4 @@
-pub use filter::{get_filenames, FileFilter, FileOptions};
+pub use filter::{collect_files, CollectOptions, FileFilter};
 pub(crate) use run::{install_hooks, run};
 
 mod filter;

diff --git a/src/cli/run/run.rs b/src/cli/run/run.rs
@@ -19,7 +19,7 @@ use constants::env_vars::EnvVars;
 
 use crate::cli::reporter::{HookInitReporter, HookInstallReporter};
 use crate::cli::run::keeper::WorkTreeKeeper;
-use crate::cli::run::{get_filenames, FileFilter, FileOptions};
+use crate::cli::run::{collect_files, CollectOptions, FileFilter};
 use crate::cli::{ExitStatus, RunExtraArgs};
 use crate::config::Stage;
 use crate::fs::Simplified;
@@ -139,7 +139,7 @@ pub(crate) async fn run(
         _guard = Some(WorkTreeKeeper::clean(&store).await?);
     }
 
-    let filenames = get_filenames(FileOptions {
+    let filenames = collect_files(CollectOptions {
         hook_stage,
         from_ref,
         to_ref,