Rename text logger

cmd/new.go

@@ -70,7 +70,7 @@ func init() {
 		"Build the changelog starting from the latest tag. Using this flag will result in a changelog with one entry.\nIt can be useful for generating a changelog to be used in release notes.",
 	)
 
-	newCmd.Flags().StringVar(&logger, "logger", "", "The type of logger to use. Valid values are 'spinner' and 'text'. The default is 'spinner'.")
+	newCmd.Flags().StringVar(&logger, "logger", "", "The type of logger to use. Valid values are 'spinner' and 'console'. The default is 'spinner'.")
 
 	newCmd.MarkFlagsMutuallyExclusive("from-version", "latest")
 	newCmd.Flags().SortFlags = false

internal/logging/console.go

@@ -0,0 +1,33 @@
+package logging
+
+import (
+	"os"
+
+	"github.com/rs/zerolog"
+	"github.com/rs/zerolog/log"
+)
+
+type consoleLogger struct {
+	loggerType LoggerType
+}
+
+func newConsoleLogger() Logger {
+	log.Logger = log.Output(zerolog.ConsoleWriter{Out: os.Stderr})
+	return &consoleLogger{
+		loggerType: CONSOLE,
+	}
+}
+
+func (c *consoleLogger) Infof(format string, args ...interface{}) {
+	log.Info().Msgf(format, args...)
+}
+
+func (c *consoleLogger) Errorf(format string, args ...interface{}) {
+	log.Error().Msgf(format, args...)
+}
+
+func (c *consoleLogger) Complete() {}
+
+func (c *consoleLogger) GetType() LoggerType {
+	return c.loggerType
+}

internal/logging/logger.go

@@ -13,7 +13,7 @@ import (
 type LoggerType int64
 
 const (
-	TEXT LoggerType = iota
+	CONSOLE LoggerType = iota
 	SPINNER
 )
 
@@ -28,8 +28,8 @@ type Logger interface {
 // NewLogger returns a new logger based on the type passed in.
 func NewLogger(loggerType LoggerType) Logger {
 	switch loggerType {
-	case TEXT:
-		return newTextLogger()
+	case CONSOLE:
+		return newConsoleLogger()
 	case SPINNER:
 		return newSpinnerLogger()
 	default:
@@ -46,18 +46,18 @@ func GetLoggerType(name string) (LoggerType, error) {
 		// If we're running in a CI environment then we don't want to
 		// use the spinner.
 		if environment.IsCI() {
-			name = "text"
+			name = "console"
 		}
 	}
 
 	var loggerType LoggerType
 	switch name {
-	case "text":
-		loggerType = TEXT
+	case "console":
+		loggerType = CONSOLE
 	case "spinner":
 		loggerType = SPINNER
 	default:
-		return loggerType, fmt.Errorf("'%s' is not a valid logger. Valid values are 'spinner' and 'text'", name)
+		return loggerType, fmt.Errorf("'%s' is not a valid logger. Valid values are 'spinner' and 'console'", name)
 	}
 
 	return loggerType, nil

internal/logging/logger_test.go

@@ -14,8 +14,8 @@ func TestNewLoggerReturnsTheCorrectType(t *testing.T) {
 		hasError bool
 	}{
 		{
-			name:     "returns a text logger",
-			want:     logging.TEXT,
+			name:     "returns a console logger",
+			want:     logging.CONSOLE,
 			hasError: false,
 		},
 		{
@@ -41,9 +41,9 @@ func TestGetLoggerTypeReturnsTheCorrectType(t *testing.T) {
 		hasError bool
 	}{
 		{
-			name:     "returns a text logger",
-			input:    "text",
-			want:     logging.TEXT,
+			name:     "returns a console logger",
+			input:    "console",
+			want:     logging.CONSOLE,
 			hasError: false,
 		},
 		{
@@ -65,7 +65,7 @@ func TestGetLoggerTypeReturnsTheCorrectType(t *testing.T) {
 			got, err := logging.GetLoggerType(tt.input)
 			if tt.hasError {
 				assert.Error(t, err)
-				assert.Equal(t, "'invalid' is not a valid logger. Valid values are 'spinner' and 'text'", err.Error())
+				assert.Equal(t, "'invalid' is not a valid logger. Valid values are 'spinner' and 'console'", err.Error())
 			} else {
 				assert.NoError(t, err)
 				assert.Equal(t, tt.want, got)