chore(vdev): filter empty strings from parsed FEATURES to avoid --features "" on FEATURES=

Author: pront

vdev/src/commands/build/vector.rs

@@ -31,7 +31,12 @@ impl Cli {
             command.arg("--release");
         }
 
-        command.features(&self.features);
+        let features: Vec<String> = self
+            .features
+            .into_iter()
+            .filter(|f| !f.is_empty())
+            .collect();
+        command.features(&features);
 
         let target = self.target.unwrap_or_else(platform::default_target);
         command.args(["--target", &target]);

vdev/src/commands/check/rust.rs

@@ -39,11 +39,17 @@ impl Cli {
             Vec::default()
         };
 
-        let feature_args = if self.no_default_features || !self.features.is_empty() {
+        let features: Vec<&str> = self
+            .features
+            .iter()
+            .map(String::as_str)
+            .filter(|f| !f.is_empty())
+            .collect();
+        let feature_args = if self.no_default_features || !features.is_empty() {
             let mut args = vec!["--no-default-features".to_string()];
-            if !self.features.is_empty() {
+            if !features.is_empty() {
                 args.push("--features".to_string());
-                args.push(self.features.join(","));
+                args.push(features.join(","));
             }
             args
         } else {

vdev/src/commands/crate_versions.rs

@@ -27,10 +27,15 @@ impl Cli {
     pub fn exec(self) -> Result<()> {
         let re_crate = Regex::new(r" (\S+) v([0-9.]+)").unwrap();
         let mut versions: HashMap<String, HashSet<String>> = HashMap::default();
+        let features: Vec<String> = self
+            .features
+            .into_iter()
+            .filter(|f| !f.is_empty())
+            .collect();
 
         for line in Command::new("cargo")
             .arg("tree")
-            .features(&self.features)
+            .features(&features)
             .check_output()?
             .lines()
         {

vdev/src/commands/test.rs

@@ -46,10 +46,16 @@ impl Cli {
             args.append(&mut extra_args);
         }
 
-        if self.no_default_features || !self.features.is_empty() {
+        let features: Vec<String> = self
+            .features
+            .into_iter()
+            .filter(|f| !f.is_empty())
+            .collect();
+
+        if self.no_default_features || !features.is_empty() {
             args.push("--no-default-features".to_string());
-            if !self.features.is_empty() {
-                args.extend(["--features".to_string(), self.features.join(",")]);
+            if !features.is_empty() {
+                args.extend(["--features".to_string(), features.join(",")]);
             }
         } else if !args.contains(&"--features".to_string()) {
             args.extend([