Update for golangci-lint 2.0

Author: aramprice

.golangci.yml

@@ -1,63 +1,12 @@
-# https://golangci-lint.run/usage/configuration/
-run:
-  timeout: 5m # 1m default times out on github-action runners
+version: "2"
 
 linters:
-  disable-all: true
-  enable:
-#    - bodyclose
-#    - dogsled
-#    - dupl
-    - errcheck
-    - errchkjson
-    - errorlint
-#    - funlen
-#    - gocognit
-    - goconst
-#    - gocritic
-#    - gocyclo
-    - goimports
-    - goprintffuncname
-    - gosec
-    - gosimple
-    - govet
-    - ineffassign
-    - makezero
-#    - misspell
-#    - musttag
-    - nakedret
-    - noctx
-    - nolintlint
-    - reassign
-#    - revive
-    - staticcheck
-#    - stylecheck
-    - typecheck
-    - unconvert
-#    - unparam
-    - unused
-#    - whitespace
-
-linters-settings:
-  gosec:
-    excludes:
-      - G115 #Potential integer overflow when converting between integer types
-#  misspell:
-#    locale: US
-  nolintlint:
-    allow-unused: true # report any unused nolint directives
-    require-explanation: false # don't require an explanation for nolint directives
-    require-specific: true # require nolint directives to be specific about which linter is being skipped
+  default: standard
 
-issues:
-  exclude-dirs:
-    - test/
-    - ".*fakes.*"
-    - ".*mocks.*"
-    - integration/
-  exclude-files:
-    - ".*_test\\.go"
+  settings:
+    errcheck:
+      check-blank: true # assignment to blank identifier: `_ := someFunc()`.
 
-output:
-  # Sort results by: filepath, line and column.
-  sort-results: true
+formatters:
+  enable:
+  - goimports

acceptance/config.go

@@ -17,7 +17,7 @@ type Config struct {
 func NewConfig(fs boshsys.FileSystem) (*Config, error) {
 	path := os.Getenv("BOSH_INIT_CONFIG_PATH")
 	if path == "" {
-		return &Config{}, errors.New("Must provide config file via BOSH_INIT_CONFIG_PATH environment variable")
+		return &Config{}, errors.New("Must provide config file via BOSH_INIT_CONFIG_PATH environment variable") //nolint:staticcheck
 	}
 
 	configContents, err := fs.ReadFile(path)
@@ -36,15 +36,15 @@ func NewConfig(fs boshsys.FileSystem) (*Config, error) {
 
 func (c *Config) Validate() error {
 	if c.StemcellPath == "" {
-		return errors.New("Must provide 'stemcell_path' in config")
+		return errors.New("Must provide 'stemcell_path' in config") //nolint:staticcheck
 	}
 
 	if c.CPIReleasePath == "" {
-		return errors.New("Must provide 'cpi_release_path' in config")
+		return errors.New("Must provide 'cpi_release_path' in config") //nolint:staticcheck
 	}
 
 	if c.DummyCompiledReleasePath == "" {
-		return errors.New("Must provide 'dummy_compiled_release_path' in config")
+		return errors.New("Must provide 'dummy_compiled_release_path' in config") //nolint:staticcheck
 	}
 
 	return nil

acceptance/instance_ssh.go

@@ -46,7 +46,7 @@ func (s *instanceSSH) setupSSH() (boshsys.File, error) {
 	success := false
 	defer func() {
 		if !success {
-			_ = s.fileSystem.RemoveAll(sshConfigFile.Name())
+			_ = s.fileSystem.RemoveAll(sshConfigFile.Name()) //nolint:errcheck
 		}
 	}()
 

acceptance/lifecycle_test.go

@@ -109,7 +109,7 @@ var _ = Describe("bosh", func() {
 	}
 
 	var deploy = func(manifestFile string, args ...string) string {
-		fmt.Fprintf(GinkgoWriter, "\n--- DEPLOY ---\n")
+		fmt.Fprintf(GinkgoWriter, "\n--- DEPLOY ---\n") //nolint:errcheck
 
 		stdout := &bytes.Buffer{}
 		multiWriter := io.MultiWriter(stdout, GinkgoWriter)
@@ -126,7 +126,7 @@ var _ = Describe("bosh", func() {
 	}
 
 	var deleteDeployment = func(manifest string, extraDeployArgs ...string) string {
-		fmt.Fprintf(GinkgoWriter, "\n--- DELETE DEPLOYMENT ---\n")
+		fmt.Fprintf(GinkgoWriter, "\n--- DELETE DEPLOYMENT ---\n") //nolint:errcheck
 
 		stdout := &bytes.Buffer{}
 		multiWriter := io.MultiWriter(stdout, GinkgoWriter)
@@ -349,7 +349,7 @@ var _ = Describe("bosh", func() {
 
 		It("fails pinging the agent", func() {
 			By("deploying with a bogus CA cert")
-			fmt.Fprintf(GinkgoWriter, "\n--- DEPLOY ---\n")
+			fmt.Fprintf(GinkgoWriter, "\n--- DEPLOY ---\n") //nolint:errcheck
 
 			stdoutBuffer := &bytes.Buffer{}
 			multiWriter := io.MultiWriter(stdoutBuffer, GinkgoWriter)

acceptance/test_environment.go

@@ -40,7 +40,7 @@ func (e testEnvironment) Path(name string) string {
 
 func (e testEnvironment) Copy(destName, srcPath string) error {
 	if srcPath == "" {
-		return fmt.Errorf("Cannot use an empty source file path '' for destination file '%s'", destName)
+		return fmt.Errorf("Cannot use an empty source file path '' for destination file '%s'", destName) //nolint:staticcheck
 	}
 
 	_, _, exitCode, err := e.cmdRunner.RunCommand(

bin/lint

@@ -15,7 +15,7 @@ export PATH=${go_bin_path}:${PATH}
 
 golangci-lint version
 
-linted_os_list=(linux)
+linted_os_list=(linux darwin)
 
 for os in "${linted_os_list[@]}"; do
   echo "lint-ing with GOOS=${os}..."

cloud/errors.go

@@ -57,13 +57,13 @@ func (e cpiError) OkToRetry() bool {
 }
 
 func mapsToNotImplementedError(method string, cmdError CmdError) bool {
-	matched, _ := regexp.MatchString("^Invalid Method:", cmdError.Message)
+	matched, _ := regexp.MatchString("^Invalid Method:", cmdError.Message) //nolint:errcheck
 
 	if cmdError.Type == "Bosh::Clouds::CloudError" && matched {
 		return true
 	}
 
-	matched, _ = regexp.MatchString("^Method is not known, got", cmdError.Message)
+	matched, _ = regexp.MatchString("^Method is not known, got", cmdError.Message) //nolint:errcheck
 
 	if cmdError.Type == "InvalidCall" && matched {
 		return true

cmd/add_blob.go

@@ -6,7 +6,7 @@ import (
 	bosherr "github.com/cloudfoundry/bosh-utils/errors"
 	boshsys "github.com/cloudfoundry/bosh-utils/system"
 
-	. "github.com/cloudfoundry/bosh-cli/v7/cmd/opts"
+	. "github.com/cloudfoundry/bosh-cli/v7/cmd/opts" //nolint:staticcheck
 	boshreldir "github.com/cloudfoundry/bosh-cli/v7/releasedir"
 	boshui "github.com/cloudfoundry/bosh-cli/v7/ui"
 )
@@ -27,7 +27,7 @@ func (c AddBlobCmd) Run(opts AddBlobOpts) error {
 		return bosherr.WrapErrorf(err, "Opening blob")
 	}
 
-	defer file.Close()
+	defer file.Close() //nolint:errcheck
 
 	blob, err := c.blobsDir.TrackBlob(opts.Args.BlobsPath, file)
 	if err != nil {

cmd/alias_env.go

@@ -2,7 +2,7 @@ package cmd
 
 import (
 	cmdconf "github.com/cloudfoundry/bosh-cli/v7/cmd/config"
-	. "github.com/cloudfoundry/bosh-cli/v7/cmd/opts"
+	. "github.com/cloudfoundry/bosh-cli/v7/cmd/opts" //nolint:staticcheck
 	boshui "github.com/cloudfoundry/bosh-cli/v7/ui"
 )
 

cmd/attach_disk.go

@@ -1,7 +1,7 @@
 package cmd
 
 import (
-	. "github.com/cloudfoundry/bosh-cli/v7/cmd/opts"
+	. "github.com/cloudfoundry/bosh-cli/v7/cmd/opts" //nolint:staticcheck
 	boshdir "github.com/cloudfoundry/bosh-cli/v7/director"
 )