chore: clippy/fmt/re-snap

Signed-off-by: Sam Gammon <[email protected]>

Author: sgammon

src/lib.rs

@@ -102,8 +102,8 @@ use directories::ProjectDirs;
 use is_terminal::IsTerminal;
 use kdl::{KdlDocument, KdlNode, KdlValue};
 use miette::{IntoDiagnostic, Result};
-use tracing::instrument;
 use oro_config::{OroConfig, OroConfigLayerExt, OroConfigOptions};
+use tracing::instrument;
 use tracing_appender::non_blocking::WorkerGuard;
 use tracing_subscriber::{
     EnvFilter,
@@ -329,10 +329,7 @@ impl Orogene {
                 .filter(|s| !s.is_empty())
                 .filter_map(|s| {
                     let dir: Result<Directive, _> = s.parse();
-                    match dir {
-                        Ok(dir) => Some(dir),
-                        Err(_) => None,
-                    }
+                    dir.ok()
                 });
             let mut filter = builder.from_env_lossy();
             for directive in directives {
@@ -516,8 +513,7 @@ impl Orogene {
 
     fn first_time_setup(&mut self) -> Result<()> {
         if let Some(config_path) = self.config.clone().or_else(|| {
-            ProjectDirs::from("", "", "orogene")
-                .map(|p| p.config_dir().to_owned().join("oro.kdl"))
+            ProjectDirs::from("", "", "orogene").map(|p| p.config_dir().to_owned().join("oro.kdl"))
         }) {
             let config_dir = config_path.parent().expect("must have parent");
             if !config_dir.exists() {

tests/snapshots/help__add.snap

@@ -39,7 +39,7 @@ Add packages as devDependencies
 
 Add packages as optionalDependencies
 
-\[aliases: optional]
+\[aliases: --optional]
 
 #### `-h, --help`
 
@@ -69,7 +69,7 @@ Whether to skip restoring packages into `node_modules` and just resolve the tree
 
 Make the resolver error if the newly-resolved tree would defer from an existing lockfile
 
-\[aliases: frozen]
+\[aliases: --frozen]
 
 #### `--no-scripts`
 
@@ -214,5 +214,3 @@ Can also be configured through the `NO_PROXY` environment variable, like `NO_PRO
 How many times to retry failed network operations
 
 \[default: 2]
-
-

tests/snapshots/help__apply.snap

@@ -49,7 +49,7 @@ Whether to skip restoring packages into `node_modules` and just resolve the tree
 
 Make the resolver error if the newly-resolved tree would defer from an existing lockfile
 
-\[aliases: frozen]
+\[aliases: --frozen]
 
 #### `--no-scripts`
 
@@ -194,5 +194,3 @@ Can also be configured through the `NO_PROXY` environment variable, like `NO_PRO
 How many times to retry failed network operations
 
 \[default: 2]
-
-

tests/snapshots/help__reapply.snap

@@ -45,7 +45,7 @@ Whether to skip restoring packages into `node_modules` and just resolve the tree
 
 Make the resolver error if the newly-resolved tree would defer from an existing lockfile
 
-\[aliases: frozen]
+\[aliases: --frozen]
 
 #### `--no-scripts`
 
@@ -190,5 +190,3 @@ Can also be configured through the `NO_PROXY` environment variable, like `NO_PRO
 How many times to retry failed network operations
 
 \[default: 2]
-
-

tests/snapshots/help__remove.snap

@@ -53,7 +53,7 @@ Whether to skip restoring packages into `node_modules` and just resolve the tree
 
 Make the resolver error if the newly-resolved tree would defer from an existing lockfile
 
-\[aliases: frozen]
+\[aliases: --frozen]
 
 #### `--no-scripts`
 
@@ -198,5 +198,3 @@ Can also be configured through the `NO_PROXY` environment variable, like `NO_PRO
 How many times to retry failed network operations
 
 \[default: 2]
-
-

tests/snapshots/help__view.snap

@@ -142,5 +142,3 @@ Can also be configured through the `NO_PROXY` environment variable, like `NO_PRO
 How many times to retry failed network operations
 
 \[default: 2]
-
-