refactor: remove unless function

Author: CPunisher

src/lib.rs

@@ -1618,10 +1618,9 @@ impl<Fs: FileSystem + Send + Sync> ResolverGeneric<Fs> {
             },
           )
           .await?;
-        TsConfig::extend_file_dependencies(
-          &mut tsconfig.file_dependencies,
-          &reference_tsconfig.file_dependencies,
-        );
+        tsconfig
+          .file_dependencies
+          .extend(reference_tsconfig.file_dependencies.iter().cloned());
         reference.tsconfig.replace(reference_tsconfig);
       }
       Ok(())

src/tests/tsconfig_project_references.rs

@@ -2,7 +2,7 @@
 
 use crate::{
   tsconfig::FileDependencies, ResolveContext, ResolveError, ResolveOptions, Resolver, ResolverPath,
-  TsConfig, TsconfigOptions, TsconfigReferences,
+  TsconfigOptions, TsconfigReferences,
 };
 
 #[test]
@@ -11,15 +11,15 @@ fn file_dependencies_are_deduplicated() {
   file_dependencies.insert("/repo/app/tsconfig.json".into());
   file_dependencies.insert("/repo/base.json".into());
 
-  let dependencies = [
+  let dependencies: FileDependencies = [
     "/repo/base.json".into(),
     "/repo/shared/tsconfig.json".into(),
     "/repo/shared/tsconfig.json".into(),
   ]
   .into_iter()
   .collect();
 
-  TsConfig::extend_file_dependencies(&mut file_dependencies, &dependencies);
+  file_dependencies.extend(dependencies.iter().cloned());
 
   assert_eq!(
     file_dependencies

src/tsconfig.rs

@@ -158,14 +158,9 @@ impl TsConfig {
         .base_url
         .clone_from(&other_config.compiler_options.base_url);
     }
-    Self::extend_file_dependencies(&mut self.file_dependencies, &other_config.file_dependencies);
-  }
-
-  pub(crate) fn extend_file_dependencies(
-    file_dependencies: &mut FileDependencies,
-    dependencies: &FileDependencies,
-  ) {
-    file_dependencies.extend(dependencies.iter().cloned());
+    self
+      .file_dependencies
+      .extend(other_config.file_dependencies.iter().cloned());
   }
 
   pub fn resolve(&self, path: &Utf8Path, specifier: &str) -> Vec<Utf8PathBuf> {