Merge pull request #30 from fishingfly/bugfix/number

fix bug: number need double quotes in annoations or labels

Author: yeahdongcn

pkg/template/string_validator.go

@@ -0,0 +1,21 @@
+package template
+
+import (
+	"strconv"
+	"strings"
+)
+
+type String string
+
+func (s String) IsBool() bool {
+	return s == "true" || s == "false"
+}
+
+func (s String) IsDecimal() bool {
+	_, err := strconv.Atoi(string(s))
+	return err == nil
+}
+
+func (s String) HasNewLine() bool {
+	return strings.Contains(string(s), "\n")
+}

pkg/template/template.go

@@ -424,9 +424,10 @@ func (p *Processor) walk(v reflect.Value, nindent int, root config.XPath, sliceI
 			p.context.setRoleNamespace = true
 		}
 		p.logger.V(10).Info("Processing others", "root", root, "s", s)
-		if s == "true" || s == "false" {
+		str := String(s)
+		if str.IsBool() || str.IsDecimal() {
 			fmt.Fprintf(p.context.out, "\"%s\"\n", v)
-		} else if strings.Contains(s, "\n") {
+		} else if str.HasNewLine() {
 			fmt.Fprintf(p.context.out, "|\n%s\n", indent(s, nindent+1))
 		} else {
 			fmt.Fprintln(p.context.out, v)