output: unexport status constants

pkg/output/json.go

@@ -44,24 +44,24 @@ func (o *jsono) Write(filename, kind, name, version string, err error, skipped b
 	s := status(kind, name, err, skipped)
 
 	switch s {
-	case VALID:
+	case statusValid:
 		st = "VALID"
 		o.nValid++
-	case INVALID:
+	case statusInvalid:
 		st = "INVALID"
 		msg = err.Error()
 		o.nInvalid++
-	case ERROR:
+	case statusError:
 		st = "ERROR"
 		msg = err.Error()
 		o.nErrors++
-	case SKIPPED:
+	case statusSkipped:
 		st = "SKIPPED"
 		o.nSkipped++
-	case EMPTY:
+	case statusEmpty:
 	}
 
-	if o.verbose || (s != VALID && s != SKIPPED && s != EMPTY) {
+	if o.verbose || (s != statusValid && s != statusSkipped && s != statusEmpty) {
 		o.results = append(o.results, result{Filename: filename, Kind: kind, Name: name, Version: version, Status: st, Msg: msg})
 	}
 

pkg/output/output.go

@@ -1,16 +1,19 @@
 package output
 
 import (
+	"fmt"
+	"os"
+
 	"github.com/yannh/kubeconform/pkg/validator"
 )
 
 const (
 	_ = iota
-	VALID
+	statusValid
-	INVALID
+	statusInvalid
-	ERROR
+	statusError
-	SKIPPED
+	statusSkipped
-	EMPTY
+	statusEmpty
 )
 
 type Output interface {
@@ -33,19 +36,19 @@ func New(outputFormat string, printSummary, verbose bool) (Output, error) {
 
 func status(kind, name string, err error, skipped bool) int {
 	if name == "" && kind == "" && err == nil && skipped == false {
-		return EMPTY
+		return statusEmpty
 	}
 
 	if skipped {
-		return SKIPPED
+		return statusSkipped
 	}
 
 	if err != nil {
 		if _, ok := err.(validator.InvalidResourceError); ok {
-			return INVALID
+			return statusInvalid
 		}
-		return ERROR
+		return statusError
 	}
 
-	return VALID
+	return statusValid
 }

pkg/output/text.go

@@ -37,27 +37,27 @@ func (o *text) Write(filename, kind, name, version string, reserr error, skipped
 
 	o.files[filename] = true
 	switch status(kind, name, reserr, skipped) {
-	case VALID:
+	case statusValid:
 		if o.verbose {
 			_, err = fmt.Fprintf(o.w, "%s - %s %s is valid\n", filename, kind, name)
 		}
 		o.nValid++
-	case INVALID:
+	case statusInvalid:
 		_, err = fmt.Fprintf(o.w, "%s - %s %s is invalid: %s\n", filename, kind, name, reserr)
 		o.nInvalid++
-	case ERROR:
+	case statusError:
 		if kind != "" && name != "" {
 			_, err = fmt.Fprintf(o.w, "%s - %s %s failed validation: %s\n", filename, kind, name, reserr)
 		} else {
 			_, err = fmt.Fprintf(o.w, "%s - failed validation: %s\n", filename, reserr)
 		}
 		o.nErrors++
-	case SKIPPED:
+	case statusSkipped:
 		if o.verbose {
 			_, err = fmt.Fprintf(o.w, "%s - %s %s skipped\n", filename, name, kind)
 		}
 		o.nSkipped++
-	case EMPTY: // sent to ensure we count the filename as parsed
+	case statusEmpty: // sent to ensure we count the filename as parsed
 	}
 
 	return err