Switch code style for meta nil removal

AliSoftware · AliSoftware · commit a3660c8f12a4 · 2022-05-24T12:18:48.000+02:00
To make it more readable See also #364 (comment)
diff --git a/lib/fastlane/plugin/wpmreleasetoolkit/helper/app_size_metrics_helper.rb b/lib/fastlane/plugin/wpmreleasetoolkit/helper/app_size_metrics_helper.rb
@@ -33,7 +33,7 @@ def meta=(hash)
       #
       def add_metric(name:, value:, meta: nil)
         metric = { name: name, value: value }
-        meta = (meta || {}).compact # Remove nil values if any
+        meta = meta&.compact || {} # Remove nil values if any (and use empty Hash if nil was provided)
         metric[:meta] = meta.map { |meta_key, meta_value| { name: meta_key.to_s, value: meta_value } } unless meta.empty?
         @metrics.append(metric)
       end

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ def meta=(hash)`
`33`	`33`	`#`
`34`	`34`	`def add_metric(name:, value:, meta: nil)`
`35`	`35`	`metric = { name: name, value: value }`
`36`		`- meta = (meta \|\| {}).compact # Remove nil values if any`
	`36`	`+ meta = meta&.compact \|\| {} # Remove nil values if any (and use empty Hash if nil was provided)`
`37`	`37`	`metric[:meta] = meta.map { \|meta_key, meta_value\| { name: meta_key.to_s, value: meta_value } } unless meta.empty?`
`38`	`38`	`@metrics.append(metric)`
`39`	`39`	`end`