Add errorlint linter (#226)

Signed-off-by: Igor Shishkin <me@teran.dev>
teran · Sep 3, 2024 · 064b6c2 · 064b6c2
1 parent dd0c06a
commit 064b6c2
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -11,6 +11,7 @@ linters:
     - err113
     - errcheck
     - errchkjson
+    - errorlint
     - errname
     - goconst
     - gocritic

diff --git a/publisher/presenter/html/handlers.go b/publisher/presenter/html/handlers.go
@@ -243,7 +243,7 @@ func (h *handlers) ErrorHandler(err error, c echo.Context) {
 	code := 500
 	templateFilename := serverErrorTemplateFilename
 
-	v, ok := err.(*echo.HTTPError)
+	v, ok := err.(*echo.HTTPError) // nolint:errorlint
 	if ok {
 		code = v.Code
 		if v.Code == http.StatusNotFound {

diff --git a/repositories/metadata/postgresql/postgresql.go b/repositories/metadata/postgresql/postgresql.go
@@ -38,7 +38,7 @@ func mapSQLErrors(err error) error {
 		return metadata.ErrNotFound
 	}
 
-	if err, ok := err.(*pq.Error); ok {
+	if err, ok := err.(*pq.Error); ok { // nolint:errorlint
 		if err.Code == "23505" {
 			return metadata.ErrConflict
 		}

diff --git a/service/service.go b/service/service.go
@@ -279,8 +279,8 @@ func (s *service) DeleteObject(ctx context.Context, namespace, container, versio
 }
 
 func mapMetadataErrors(err error) error {
-	switch err {
-	case metadata.ErrNotFound:
+	switch {
+	case errors.Is(err, metadata.ErrNotFound):
 		return ErrNotFound
 	default:
 		return err