apply cargo clippy

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "scoop-search"
-version = "0.5.7"
+version = "0.5.8"
 authors = ["tokiedokie <42903136+tokiedokie@users.noreply.github.com>"]
 edition = "2018"
 

src/app.rs

@@ -110,10 +110,7 @@ impl App {
 
         let filtered: Vec<App> = tree
             .as_array()
-            .ok_or_else(|| Box::<dyn Error>::from(format!(
-                "{} key `tree` is invalid",
-                remote_url
-            )))?
+            .ok_or_else(|| Box::<dyn Error>::from(format!("{} key `tree` is invalid", remote_url)))?
             .iter()
             .map(|obj| obj["path"].as_str().unwrap_or("").to_string())
             .filter(|path| path.ends_with(".json"))

src/bucket.rs

@@ -77,14 +77,14 @@ impl Bucket {
         let mut app_in_local = false;
 
         for bucket_path in bucket_paths {
-            let bucket_name = Bucket::get_name(&bucket_path).unwrap_or(String::new());
+            let bucket_name = Bucket::get_name(&bucket_path).unwrap_or_default();
             let app_paths = App::get_app_paths(&bucket_path)?;
 
             let apps: Vec<App> = app_paths.iter().map(|path| App::new(path)).collect();
 
             let filtered_apps = App::search_apps(&apps, query);
 
-            if filtered_apps.len() > 0 {
+            if !filtered_apps.is_empty() {
                 app_in_local = true;
                 crate::display_apps(&bucket_name, &filtered_apps);
             }

src/scoop.rs

@@ -41,7 +41,7 @@ impl Scoop {
         //Ok(config.get("rootPath").is_some())
         Ok(config
             .get("rootPath")
-            .ok_or(Box::<dyn Error>::from(
+            .ok_or_else(|| Box::<dyn Error>::from(
                 "Can't get rootPath in scoop/config.json",
             ))?
             .to_string())