make lint happy

apache · Nov 8, 2023 · 82b0a8a · 82b0a8a
1 parent 3d7bbc0
commit 82b0a8a
Show file tree

Hide file tree

Showing 10 changed files with 11 additions and 15 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -36,11 +36,6 @@ linters-settings:
   goconst:
     min-len: 2
     min-occurrences: 2
-  depguard:
-    list-type: blacklist
-    include-go-root: true
-    packages-with-error-messages:
-      fmt: "logging is allowed only by logutils.Log"
   misspell:
     locale: US
     ignore-words:
@@ -98,7 +93,6 @@ linters-settings:
 linters:
   enable:
     - deadcode
-    - depguard
     - dogsled
     - dupl
     - errcheck
@@ -115,7 +109,6 @@ linters:
     - ineffassign
     - lll
     - misspell
-    - nakedret
     - staticcheck
     - stylecheck
     - typecheck
@@ -125,6 +118,8 @@ linters:
     - whitespace
   disable:
     - structcheck
+    - depguard
+    - nakedret
 
 service:
   golangci-lint-version: 1.20.x

diff --git a/Makefile b/Makefile
@@ -40,7 +40,7 @@ all: clean lint test build
 
 .PHONY: lint
 lint:
-	$(GO_LINT) version || curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(GO_PATH)/bin -d "v1.46.2"
+	$(GO_LINT) version || curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(GO_PATH)/bin -d "v1.55.2"
 	$(GO_LINT) run -v --timeout 5m ./...
 
 .PHONY: fix-lint

diff --git a/cmd/e2e/main.go b/cmd/e2e/main.go
@@ -14,7 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
-//
+
 package main
 
 import (

diff --git a/commands/cleanup/cleanup.go b/commands/cleanup/cleanup.go
@@ -1,4 +1,3 @@
-//
 // Licensed to Apache Software Foundation (ASF) under one or more contributor
 // license agreements. See the NOTICE file distributed with
 // this work for additional information regarding copyright
@@ -15,6 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
+
 package cleanup
 
 import (

diff --git a/commands/root.go b/commands/root.go
@@ -14,7 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
-//
+
 package commands
 
 import (

diff --git a/commands/run/run.go b/commands/run/run.go
@@ -1,4 +1,3 @@
-//
 // Licensed to Apache Software Foundation (ASF) under one or more contributor
 // license agreements. See the NOTICE file distributed with
 // this work for additional information regarding copyright
@@ -15,6 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
+
 package run
 
 import (

diff --git a/commands/trigger/trigger.go b/commands/trigger/trigger.go
@@ -1,4 +1,3 @@
-//
 // Licensed to Apache Software Foundation (ASF) under one or more contributor
 // license agreements. See the NOTICE file distributed with
 // this work for additional information regarding copyright
@@ -15,6 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
+
 package trigger
 
 import (

diff --git a/commands/version.go b/commands/version.go
@@ -14,7 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
-//
+
 package commands
 
 var version string
diff --git a/internal/components/trigger/http.go b/internal/components/trigger/http.go
@@ -15,6 +15,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
+
 package trigger
 
 import (

diff --git a/internal/logger/log.go b/internal/logger/log.go
@@ -14,7 +14,7 @@
 // KIND, either express or implied.  See the License for the
 // specific language governing permissions and limitations
 // under the License.
-//
+
 package logger
 
 import (