Merge pull request #8 from Invoca/TECH-17815-allow-multiple-overrides

TECH-17815 Allow multiple overrides

Author: jebentier

CHANGELOG.md

@@ -4,6 +4,10 @@ Inspired by [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.3] - 2025-03-25
+### Fixed
+- Fixed `--variable-override` to accept multiple arguments to override multiple variables
+
 ## [0.2.2] - 2024-06-17
 ### Fixed
 - Fixed a bug allowing child process errors to be ignored while rendering.

lib/kubernetes_template_rendering/cli.rb

@@ -34,9 +34,12 @@ def parse(options)
           op.on("--[no-]prune",                                "enable/disable pruning of untouched resources")                       { args.prune = _1 }
           op.on("--source-repo=SOURCE_REPO",                   "set the source repo for the rendered templates")                      { args.source_repo = _1 }
 
-          op.on("--variable-override=KEY:VALUE", "override a variable value set within definitions.yaml") do |override|
-            args.variable_overrides ||= {}
-            args.variable_overrides.merge!(Hash[[override.split(":", 2)]])
+          op.on("--variable-override=KEY:VALUE", "override a variable value set within definitions.yaml", Array) do |overrides|
+            args.variable_overrides ||= {} # Initialize as a Hash
+            overrides.each do |override|
+              key, value = override.split(":", 2)
+              args.variable_overrides[key] = value if key && value
+            end
           end
 
           op.on("-h", "--help") do

lib/kubernetes_template_rendering/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module KubernetesTemplateRendering
-  VERSION = "0.2.2"
+  VERSION = "0.2.3"
 end