minor nits

Makefile

@@ -1,5 +1,7 @@
 #!/usr/bin/make -f
 
+.PHONY: test-build test build build-static docker-test docker-build-static
+
 test-build: test build
 
 test:

pkg/output/main.go

@@ -5,10 +5,10 @@ import (
 )
 
 const (
-	VALID   = iota
+	VALID = iota
-	INVALID = iota
+	INVALID
-	ERROR   = iota
+	ERROR
-	SKIPPED = iota
+	SKIPPED
 )
 
 type Output interface {

pkg/registry/local.go

@@ -56,7 +56,5 @@ func (r LocalSchemas) DownloadSchema(resourceKind, resourceAPIVersion, k8sVersio
 		return nil, fmt.Errorf("failed to open schema %s", schemaFile)
 	}
 	defer f.Close()
-	content, err := ioutil.ReadAll(f)
+	return ioutil.ReadAll(f)
-
-	return content, err
 }

pkg/resource/main_test.go

@@ -10,8 +10,8 @@ func TestSignatureFromBytes(t *testing.T) {
 		name string
 		have []byte
 		want resource.Signature
-		err error
+		err  error
-	} {
+	}{
 		{
 			name: "valid deployment",
 			have: []byte(`
@@ -24,10 +24,10 @@ metadata:
     app: myService
 spec:
 `),
-			want: resource.Signature {
+			want: resource.Signature{
-					Kind: "Deployment",
+				Kind:      "Deployment",
-					Version: "apps/v1",
+				Version:   "apps/v1",
-					Namespace: "default",
+				Namespace: "default",
 			},
 			err: nil,
 		},
@@ -45,4 +45,4 @@ spec:
 		}
 
 	}
-}
+}

pkg/validator/main.go

@@ -45,16 +45,16 @@ func Validate(rawResource []byte, schema *gojsonschema.Schema) error {
 		return fmt.Errorf("problem validating schema. Check JSON formatting: %s", err)
 	}
 
-	if !results.Valid() {
+	if results.Valid() {
-		msg := ""
+		return nil
-		for _, errMsg := range results.Errors() {
-			if msg != "" {
-				msg += " - "
-			}
-			msg += errMsg.Description()
-		}
-		return InvalidResourceError{err: msg}
 	}
 
-	return nil
+	msg := ""
+	for _, errMsg := range results.Errors() {
+		if msg != "" {
+			msg += " - "
+		}
+		msg += errMsg.Description()
+	}
+	return InvalidResourceError{err: msg}
 }