fix: lint errors

Jujulego · Jun 9, 2024 · d384ad2 · d384ad2
1 parent 1f5a32e
commit d384ad2
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 10 deletions.
diff --git a/crates/ring-cli/src/js/list.rs b/crates/ring-cli/src/js/list.rs
@@ -34,7 +34,7 @@ pub fn handle_command(args: &ArgMatches) -> Result<()> {
         for workspace in project.list_workspaces()? {
             println!(" - {workspace}");
         }
-    } else if let None = JsProject::search_from(&project_dir)? {
+    } else {
         warn!("Project root not found");
     }
 

diff --git a/crates/ring-js-project/src/package_manager.rs b/crates/ring-js-project/src/package_manager.rs
@@ -1,17 +1,12 @@
 use std::fmt::{Display, Formatter};
 
-#[derive(Debug, Copy, Clone, Eq, PartialEq)]
+#[derive(Debug, Default, Copy, Clone, Eq, PartialEq)]
 pub enum PackageManager {
+    #[default]
     NPM,
     Yarn
 }
 
-impl Default for PackageManager {
-    fn default() -> Self {
-        PackageManager::NPM
-    }
-}
-
 impl Display for PackageManager {
     fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
         match self {

diff --git a/crates/ring-js-project/src/project.rs b/crates/ring-js-project/src/project.rs
@@ -1,5 +1,5 @@
 use anyhow::{Context, Result};
-use std::path::{Path, PathBuf};
+use std::path::Path;
 use glob::glob;
 use tracing::{debug, trace};
 use crate::constants::{LOCKFILES, MANIFEST};
@@ -98,6 +98,6 @@ impl JsProject {
     }
 
     pub fn get_root(&self) -> &Path {
-        &self.main_workspace.get_root()
+        self.main_workspace.get_root()
     }
 }