Merge pull request #73 from DeterminateSystems/fixup-empty-strings-fr…

…om-env
DeterminateSystems · Aug 30, 2023 · 9e7edd8 · 9e7edd8
2 parents 3233157 + b089128
commit 9e7edd8
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/cli/mod.rs b/src/cli/mod.rs
@@ -263,11 +263,14 @@ impl NixfrPushCli {
             mirror,
             jwt_issuer_uri,
             instrumentation: _,
-            mut extra_labels,
+            extra_labels,
             spdx_expression,
             extra_tags,
         } = self;
 
+        let mut extra_labels: Vec<_> = extra_labels.into_iter().filter(|v| !v.is_empty()).collect();
+        let extra_tags: Vec<_> = extra_tags.into_iter().filter(|v| !v.is_empty()).collect();
+
         let is_github_actions = std::env::var("GITHUB_ACTION").ok().is_some();
         if !extra_tags.is_empty() {
             let message = "`extra-tags` is deprecated and will be removed in the future. Please use `extra-labels` instead.";