refactor: move is_package_file to Detector impl`

Signed-off-by: Luke-zhang-04 <[email protected]>

Author: Luke-zhang-04

Diff for: src/onefetch/deps/mod.rs

@@ -40,13 +40,13 @@ impl DependencyDetector {
         DependencyDetector { package_managers }
     }
 
-    pub fn get_deps_info(&self, dir: &str) -> Result<String> {
-        fn is_package_file(detector: &DependencyDetector, file_name: &str) -> bool {
-            detector.package_managers.iter().any(|(package_manager_file_name, _)| {
-                file_name.starts_with(package_manager_file_name)
-            })
-        }
+    fn is_package_file(&self, file_name: &str) -> bool {
+        self.package_managers.iter().any(|(package_manager_file_name, _)| {
+            file_name.starts_with(package_manager_file_name)
+        })
+    }
 
+    pub fn get_deps_info(&self, dir: &str) -> Result<String> {
         let deps = fs::read_dir(dir)
             .chain_err(|| "Could not read directory")?
             .filter_map(std::result::Result::ok)
@@ -56,7 +56,7 @@ impl DependencyDetector {
                     && entry
                         .file_name()
                         .map(OsStr::to_string_lossy)
-                        .map(|s| is_package_file(&self, s.as_ref()))
+                        .map(|s| self.is_package_file(s.as_ref()))
                         .unwrap_or_default()
             })
             .map(|entry| {