Merge pull request #166 from Screenly/feature/ci_env_var

Updates: just checks if CI env is defined.
Screenly · May 15, 2024 · e5c45d5 · e5c45d5
2 parents 7e3fe8c + 94fd472
commit e5c45d5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/commands/edge_app.rs b/src/commands/edge_app.rs
@@ -405,7 +405,7 @@ impl EdgeAppCommand {
         actual_app_id: String,
         changed_settings: SettingChanges,
     ) -> Result<(), CommandError> {
-        match { delete_missing_settings } {
+        match delete_missing_settings  {
             Some(delete) => {
                 if delete {
                     self.delete_deleted_settings(
@@ -416,10 +416,8 @@ impl EdgeAppCommand {
                 }
             }
             None => {
-                if let Ok(ci) = std::env::var("CI") {
-                    if ci == "true" {
-                        return Ok(());
-                    }
+                if let Ok(_ci) = std::env::var("CI") {
+                    return Ok(());
                 }
                 self.delete_deleted_settings(
                     actual_app_id.clone(),

diff --git a/src/commands/edge_app_server.rs b/src/commands/edge_app_server.rs
@@ -80,7 +80,7 @@ pub async fn run_server(
 }
 
 #[derive(Debug)]
-struct WarpError(anyhow::Error);
+struct WarpError(#[allow(dead_code)] anyhow::Error);
 
 impl Reject for WarpError {}