refactor: dead code removed, methods renamed

src/parse_args.rs

@@ -65,12 +65,7 @@ impl CliArgs {
     /// An ENV is set in the ./containerised/Dockerfile, if this is ENV found, then sleep for 250ms, else the container, for as yet unknown reasons, will close immediately
     /// returns a bool, so that the `update_all_containers()` won't bother to check the entry point unless running via a container
     fn check_if_in_container() -> bool {
-        if let Ok(value) = std::env::var(ENV_KEY) {
-            if value == ENV_VALUE {
-                return true;
-            }
-        }
-        false
+        std::env::var(ENV_KEY).map_or(false, |i| i == ENV_VALUE)
     }
 
     /// Parse cli arguments