- update golangci-lint to 1.52.1 to support Go 1.20 (See golangci/gol…

…angci-lint#3414)

- ignore some errors, should refactor in the future

Signed-off-by: Ivan Mikheykin <ivan.mikheykin@flant.com>

.golangci.yaml

@@ -46,3 +46,7 @@ issues:
   # Show all errors.
   max-issues-per-linter: 0
   max-same-issues: 0
+
+  exclude:
+    # TODO use %w in the future.
+    - "non-wrapping format verb for fmt.Errorf" # errorlint

Taskfile.dist.yaml

@@ -509,7 +509,7 @@ tasks:
   deps:install:golangci-lint:
     desc: "Install golangci-lint binary."
     cmds:
-      - curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin {{.golangcilintVersion | default "v1.50.0"}}
+      - curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin {{.golangcilintVersion | default "v1.52.2"}}
 
   deps:install:prettier:
     desc: "Install prettier."

pkg/buildah/common.go

@@ -256,7 +256,7 @@ func GetDefaultIsolation() (thirdparty.Isolation, error) {
 	return thirdparty.IsolationChroot, nil
 }
 
-//nolint:unused
+//nolint:nolintlint,unused
 func debug() bool {
 	return os.Getenv("WERF_BUILDAH_DEBUG") == "1"
 }

pkg/docker_registry/docker_hub.go

@@ -93,6 +93,8 @@ func (r *dockerHub) DeleteRepoImage(ctx context.Context, repoImage *image.Info)
 		return err
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	resp, err := r.dockerHubApi.deleteTag(ctx, account, project, repoImage.Tag, token)
 	if err != nil {
 		return r.handleFailedApiResponse(resp, err)
@@ -112,6 +114,8 @@ func (r *dockerHub) deleteRepo(ctx context.Context, reference string) error {
 		return err
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	resp, err := r.dockerHubApi.deleteRepository(ctx, account, project, token)
 	if err != nil {
 		return r.handleFailedApiResponse(resp, err)
@@ -125,6 +129,8 @@ func (r *dockerHub) getToken(ctx context.Context) (string, error) {
 		return r.dockerHubCredentials.token, nil
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	token, resp, err := r.dockerHubApi.getToken(ctx, r.dockerHubCredentials.username, r.dockerHubCredentials.password)
 	if err != nil {
 		return "", r.handleFailedApiResponse(resp, err)

pkg/docker_registry/github_packages.go

@@ -102,13 +102,17 @@ func (r *gitHubPackages) DeleteRepoImage(ctx context.Context, repoImage *image.I
 	}
 
 	if isUser {
+		//nolint:bodyclose
+		// TODO: close response body
 		if resp, err := r.gitHubApi.deleteUserContainerPackageVersion(ctx, packageName, packageVersionId, r.token); err != nil {
 			return r.handleFailedApiResponse(resp, err)
 		}
 
 		return nil
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	if resp, err := r.gitHubApi.deleteOrgContainerPackageVersion(ctx, orgOrUserName, packageName, packageVersionId, r.token); err != nil {
 		return r.handleFailedApiResponse(resp, err)
 	}
@@ -128,13 +132,17 @@ func (r *gitHubPackages) DeleteRepo(ctx context.Context, reference string) error
 	}
 
 	if isUser {
+		//nolint:bodyclose
+		// TODO: close response body
 		if resp, err := r.gitHubApi.deleteUserContainerPackage(ctx, packageName, r.token); err != nil {
 			return r.handleFailedApiResponse(resp, err)
 		}
 
 		return nil
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	if resp, err := r.gitHubApi.deleteOrgContainerPackage(ctx, orgOrUserName, packageName, r.token); err != nil {
 		return r.handleFailedApiResponse(resp, err)
 	}
@@ -178,13 +186,17 @@ func (r *gitHubPackages) populateTagIDPackageVersionIDCache(ctx context.Context,
 	}
 
 	if isUser {
+		//nolint:bodyclose
+		// TODO: close response body
 		if resp, err := r.gitHubApi.getUserContainerPackageVersionsInBatches(ctx, packageName, r.token, handleFunc); err != nil {
 			return r.handleFailedApiResponse(resp, err)
 		}
 
 		return nil
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	if resp, err := r.gitHubApi.getOrgContainerPackageVersionsInBatches(ctx, orgOrUserName, packageName, r.token, handleFunc); err != nil {
 		return r.handleFailedApiResponse(resp, err)
 	}
@@ -202,6 +214,8 @@ func (r *gitHubPackages) isUser(ctx context.Context, orgOrUserName string) (bool
 		return isUser.(bool), nil
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	user, resp, err := r.gitHubApi.getUser(ctx, orgOrUserName, r.token)
 	if err != nil {
 		return false, r.handleFailedApiResponse(resp, err)

pkg/docker_registry/selectel.go

@@ -110,6 +110,8 @@ func (r *selectel) deleteRepoImage(ctx context.Context, repoImage *image.Info) e
 		return err
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	resp, err := r.selectelApi.deleteReference(ctx, hostname, registryID, repository, repoImage.RepoDigest, token)
 	if err != nil {
 		return r.handleFailedApiResponse(resp, err)
@@ -129,6 +131,8 @@ func (r *selectel) deleteRepo(ctx context.Context, reference string) error {
 		return err
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	resp, err := r.selectelApi.deleteRepository(ctx, hostname, registryID, repository, token)
 	if err != nil {
 		return r.handleFailedApiResponse(resp, err)
@@ -148,6 +152,8 @@ func (r *selectel) tags(ctx context.Context, reference string) ([]string, error)
 		return nil, err
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	tags, resp, err := r.selectelApi.getTags(ctx, hostname, registryID, repository, token)
 	if err != nil {
 		return nil, r.handleFailedApiResponse(resp, err)
@@ -195,6 +201,8 @@ func (r *selectel) getRegistryId(ctx context.Context, token, reference string) (
 		return "", err
 	}
 
+	//nolint:bodyclose
+	// TODO: close response body
 	registryId, resp, err := r.selectelApi.getRegistryId(ctx, hostname, registry, token)
 	if err != nil {
 		return "", r.handleFailedApiResponse(resp, err)