Enable linters for golangci-lint

Enabled linters: prealloc, scopelint, typecheck, unconvert Only "unconvert" linter had an issue in pkg/scaffold/project/project_test.go whish is fixed.
kubernetes-sigs · Jan 27, 2020 · 41765a3 · 41765a3
1 parent cf950d8
commit 41765a3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/pkg/scaffold/project/project_test.go b/pkg/scaffold/project/project_test.go
@@ -133,7 +133,7 @@ Copyright %s Example Owners.
 			})
 
 			It("should skip writing Gopkg.toml", func() {
-				e := strings.Replace(string(result.Golden), project.DefaultGopkgHeader, "", -1)
+				e := strings.Replace(result.Golden, project.DefaultGopkgHeader, "", -1)
 				_, err = f.Write([]byte(e))
 				Expect(err).NotTo(HaveOccurred())
 				Expect(f.Close()).NotTo(HaveOccurred())
@@ -158,7 +158,7 @@ Copyright %s Example Owners.
 			})
 
 			It("should keep the user content", func() {
-				e := strings.Replace(string(result.Golden),
+				e := strings.Replace(result.Golden,
 					project.DefaultGopkgUserContent, "Hello World", -1)
 				_, err = f.Write([]byte(e))
 				Expect(err).NotTo(HaveOccurred())

diff --git a/scripts/verify.sh b/scripts/verify.sh
@@ -37,8 +37,12 @@ golangci-lint run --disable-all --deadline 5m \
     --enable=maligned \
     --enable=misspell \
     --enable=nakedret \
+    --enable=prealloc \
+    --enable=scopelint \
     --enable=staticcheck \
     --enable=structcheck \
+    --enable=typecheck \
+    --enable=unconvert \
     --enable=unparam \
     --enable=unused \
     --enable=varcheck \