tsk-82: Refactor validate_args() method (#90)

* tsk-82: Refactored validate_flags() method

* tsk-82: Warning fix

* tsk-82: Code formatting

* tsk-82: Version bump

* tsk-82: Forgot this

* tsk-82: Fix

Cargo.lock

@@ -451,7 +451,7 @@ dependencies = [
 
 [[package]]
 name = "icarus-dm"
-version = "0.8.7"
+version = "0.8.8"
 dependencies = [
  "futures",
  "http",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "icarus-dm"
-version = "0.8.7"
+version = "0.8.8"
 rust-version = "1.90"
 edition = "2024"
 

src/managers/action_managers.rs

@@ -68,17 +68,17 @@ impl ActionManager {
 
         while i < flag_vals.len() {
             let flag = &flag_vals[i];
-
             let mut flg = models::flags::Flags::default();
 
-            // TODO: Refactor this
-            if self.is_valid_flag(flag) && self.does_flag_have_value(flag) {
+            if self.is_valid_flag(flag) {
+                if self.does_flag_have_value(flag) {
                     flg.flag = String::from(flag);
                     flg.value = String::from(&flag_vals[i + 1]);
 
                     i += 1;
-            } else if self.is_valid_flag(flag) {
+                } else {
                     flg.flag = String::from(flag);
+                }
             } else {
                 utilities::checks::exit_program(-1);
             }

src/managers/commit_manager.rs

@@ -1,7 +1,7 @@
 use std::collections::HashMap;
 use std::default::Default;
 use std::fs::read_dir;
-use std::io::{Result, Write};
+use std::io::Result;
 use std::str::FromStr;
 
 use crate::managers;