Merge pull request #899 from wyardley/wyardley/config-values-again

Additional formatting updates for config validation

config/commands.go

@@ -3,16 +3,27 @@ package config
 import (
 	"fmt"
 	"os"
+	"sort"
 	"strings"
 
 	"github.com/pkg/errors"
 	"gopkg.in/yaml.v3"
 )
 
 func printValues(values Values) {
-	for key, value := range values {
-		fmt.Fprintf(os.Stderr, "%-18s %s\n", key+":", value)
+	// Provide a stable sort order for printed values
+	keys := make([]string, 0, len(values))
+	for k := range values {
+		keys = append(keys, k)
 	}
+	sort.Strings(keys)
+
+	for _, key := range keys {
+		fmt.Fprintf(os.Stderr, "%-18s %v\n", key+":", values[key])
+	}
+
+	// Add empty newline at end
+	fmt.Fprintf(os.Stderr, "\n")
 }
 
 type ProcessConfigOpts struct {
@@ -73,7 +84,7 @@ func (c *ConfigCompiler) ProcessConfig(opts ProcessConfigOpts) error {
 	//if no orgId provided use org slug
 	values := LocalPipelineValues()
 	if opts.VerboseOutput {
-		fmt.Println("Processing config with following values")
+		fmt.Fprintln(os.Stderr, "Processing config with following values:")
 		printValues(values)
 	}
 
@@ -118,7 +129,7 @@ func (c *ConfigCompiler) ValidateConfig(opts ValidateConfigOpts) error {
 	//if no orgId provided use org slug
 	values := LocalPipelineValues()
 	if opts.VerboseOutput {
-		fmt.Println("Validating config with following values")
+		fmt.Fprintln(os.Stderr, "Validating config with following values:")
 		printValues(values)
 	}
 
@@ -152,6 +163,6 @@ func (c *ConfigCompiler) ValidateConfig(opts ValidateConfigOpts) error {
 		}
 	}
 
-	fmt.Printf("\nConfig file at %s is valid.\n", opts.ConfigPath)
+	fmt.Printf("Config file at %s is valid.\n", opts.ConfigPath)
 	return nil
 }