clippy: use inspect when map returns the original item (#98)

openSUSE-Rust · Sep 10, 2024 · 7a62324 · 7a62324
1 parent 2dbe19f
commit 7a62324
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/bulk-updater/src/operations.rs b/bulk-updater/src/operations.rs
@@ -264,12 +264,11 @@ pub fn attempt_cargo_update_before_revendor(
         "🔼 Attempting to update in progress at {}",
         package_path.to_string_lossy()
     );
-    let cargo_vendor_params = does_have_cargo_vendor(package_path).map_err(|err| {
+    let cargo_vendor_params = does_have_cargo_vendor(package_path).inspect_err(|_| {
         tracing::error!(
             "❌ Error -- {} is not setup for cargo vendor!",
             package_path.to_string_lossy()
-        );
-        err
+        )
     })?;
     tracing::info!(
         ?cargo_vendor_params,

diff --git a/cargo/src/utils/mod.rs b/cargo/src/utils/mod.rs
@@ -338,10 +338,7 @@ pub fn process_globs(src: &Path) -> io::Result<PathBuf> {
     // Take the last item.
     globs
         .pop()
-        .map(|item| {
-            info!("🍿 Vendoring for src '{}'", item.display());
-            item
-        })
+        .inspect(|item| info!("🍿 Vendoring for src '{}'", item.display()))
         .ok_or_else(|| {
             error!("No files/directories matched src glob input");
             io::Error::new(