update golangci-lint

.github/workflows/golang-ci-lint.yml

@@ -11,18 +11,8 @@ jobs:
     name: lint
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v6
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v2
+        uses: golangci/golangci-lint-action@v9.0.0
         with:
-          # Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
-          version: v1.29
-
-          # Optional: working directory, useful for monorepos
-          # working-directory: somedir
-
-          # Optional: golangci-lint command line arguments.
-          # args: --issues-exit-code=0
-
-          # Optional: show only new issues if it's a pull request. The default value is `false`.
-          # only-new-issues: true
+          version: latest

google/creds.go

@@ -20,7 +20,7 @@ func (cm *credentialsManager) loadCreds(c *Client) error {
 	if err != nil {
 		return err
 	}
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 	var token oauth2.Token
 	if err := json.NewDecoder(f).Decode(&token); err != nil {
 		return err
@@ -34,7 +34,7 @@ func (cm *credentialsManager) saveCreds(c *Client) error {
 	if err != nil {
 		return err
 	}
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 	return json.NewEncoder(f).Encode(c.credentials)
 }
 

google/google.go

@@ -51,7 +51,7 @@ func NewClientFromFile(logger *log.Logger, path string, options ...ClientOption)
 	if err != nil {
 		return nil, err
 	}
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 	return NewClientFromReader(logger, f, options...)
 }
 

main.go

@@ -198,7 +198,7 @@ func NewConfigFromFile(logger *log.Logger, path string, googleClient *google.Cli
 	if f == nil {
 		r = bytes.NewReader(nil)
 	} else {
-		defer f.Close()
+		defer f.Close() //nolint:errcheck
 	}
 
 	return NewConfigFromReader(logger, r, googleClient, zoomClient, slackClient)
@@ -436,7 +436,7 @@ func (z *Config) Archive(ctx context.Context, meeting zoom.Meeting, params runPa
 			curArchMeeting.status = "error"
 			return fmt.Errorf("while downloading recording %s: %w", f.DownloadURL, err)
 		}
-		defer r.Body.Close()
+		defer r.Body.Close() //nolint:errcheck
 
 		if r.StatusCode != http.StatusOK {
 			curArchMeeting.status = "error"

slack/config.go

@@ -18,7 +18,7 @@ func NewClientFromFile(logger *log.Logger, path string, options ...slackapi.Opti
 	if err != nil {
 		return nil, err
 	}
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 	return NewClientFromReader(logger, f, options...)
 }
 

zoom/creds.go

@@ -20,7 +20,7 @@ func (cm *credentialsManager) loadCreds(c *Client) error {
 	if err != nil {
 		return err
 	}
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 	var token oauth2.Token
 	if err := json.NewDecoder(f).Decode(&token); err != nil {
 		return err
@@ -34,7 +34,7 @@ func (cm *credentialsManager) saveCreds(c *Client) error {
 	if err != nil {
 		return err
 	}
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 	return json.NewEncoder(f).Encode(c.credentials)
 }
 

zoom/zoom.go

@@ -93,7 +93,7 @@ func NewClientFromFile(logger *log.Logger, path string, options ...ClientOption)
 	if err != nil {
 		return nil, err
 	}
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 	return NewClientFromReader(logger, f, options...)
 }
 
@@ -210,7 +210,7 @@ func (c *Client) Do(req *http.Request, decodeTo interface{}) (*http.Response, er
 	if err != nil {
 		return nil, fmt.Errorf("while creating http client in Do: %w", err)
 	}
-	defer rsp.Body.Close()
+	defer rsp.Body.Close() //nolint:errcheck
 	if rsp.StatusCode != http.StatusOK {
 		msg := fmt.Sprintf("API call to %s failed: %d", req.URL.String(), rsp.StatusCode)
 		if body, err := ioutil.ReadAll(rsp.Body); err == nil {