From 13275c775706f0d7e7c808c55f6cea00ec19984a Mon Sep 17 00:00:00 2001 From: Maya Date: Tue, 18 Aug 2026 16:11:55 +0200 Subject: [PATCH] Add `expect` msg over undocumented unwraps --- src/symlinks.rs | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/symlinks.rs b/src/symlinks.rs index 5ac2416..ad4d827 100644 --- a/src/symlinks.rs +++ b/src/symlinks.rs @@ -602,7 +602,9 @@ pub fn add_cmd( let Some(group_files) = group else { return }; for file in group_files { - let target_file = file.to_target_path().unwrap(); + let target_file = file + .to_target_path() + .expect("Unable to convert file to a target path?"); let deleted_file = if adopt { &file.path } else { &target_file }; @@ -612,9 +614,9 @@ pub fn add_cmd( t!("dry-run.removing_x", x = deleted_file.display()).red() ); } else if deleted_file.is_dir() { - fs::remove_dir_all(deleted_file).unwrap(); + fs::remove_dir_all(deleted_file).expect("Unable to delete directory?"); } else if deleted_file.is_file() { - fs::remove_file(deleted_file).unwrap(); + fs::remove_file(deleted_file).expect("Unable to remove file?"); } if adopt { @@ -629,7 +631,7 @@ pub fn add_cmd( .yellow() ); } else { - fs::rename(target_file, &file.path).unwrap(); + fs::rename(target_file, &file.path).expect("Unable to rename (replace) file?"); } } }