Merge pull request #607 from jzelinskie/gofmt

enforce gofmt in travis

.travis.yml

@@ -1,7 +1,7 @@
 language: go
 
 go: 
-  - "1.10"
+  - "1.11"
 
 sudo: required
 
@@ -16,6 +16,7 @@ install:
 - chmod +x $HOME/.local/bin/prototool
 
 script:
+- diff -u <(echo -n) <(gofmt -l -s $(go list -f '{{.Dir}}') | grep -v '/vendor/')
 - prototool format -d api/v3/clairpb/clair.proto
 - prototool lint api/v3/clairpb/clair.proto
 - go test $(glide novendor | grep -v contrib)

ext/featurens/driver_test.go

@@ -45,8 +45,8 @@ func TestMultipleNamespaceDetector(t *testing.T) {
 	testData := []MultipleNamespaceTestData{
 		{
 			ExpectedNamespaces: []database.Namespace{
-				database.Namespace{Name: "debian:8", VersionFormat: "dpkg"},
-				database.Namespace{Name: "alpine:v3.3", VersionFormat: "dpkg"},
+				{Name: "debian:8", VersionFormat: "dpkg"},
+				{Name: "alpine:v3.3", VersionFormat: "dpkg"},
 			},
 			Files: tarutil.FilesMap{
 				"etc/os-release": []byte(`

ext/vulnsrc/rhel/rhel.go

@@ -372,7 +372,7 @@ func severity(def definition) database.Severity {
 	case "Critical":
 		return database.CriticalSeverity
 	default:
-		log.Warning("could not determine vulnerability severity from: %s.", def.Title)
+		log.Warningf("could not determine vulnerability severity from: %s.", def.Title)
 		return database.UnknownSeverity
 	}
 }

ext/vulnsrc/ubuntu/ubuntu.go

@@ -416,7 +416,7 @@ func SeverityFromPriority(priority string) database.Severity {
 	case "critical":
 		return database.CriticalSeverity
 	default:
-		log.Warning("could not determine a vulnerability severity from: %s", priority)
+		log.Warningf("could not determine a vulnerability severity from: %s", priority)
 		return database.UnknownSeverity
 	}
 }

updater_test.go

@@ -132,7 +132,7 @@ func newmockUpdaterDatastore() *mockUpdaterDatastore {
 				vuln, ok := session.copy.vulnerabilities[id]
 				r = append(r, database.NullableVulnerability{
 					VulnerabilityWithAffected: vuln,
-					Valid: ok,
+					Valid:                     ok,
 				})
 			}
 			return r, nil