diff --git a/.github/workflows/golang-ci-lint.yml b/.github/workflows/golang-ci-lint.yml index 84df9ad..d8b898d 100644 --- a/.github/workflows/golang-ci-lint.yml +++ b/.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 diff --git a/google/creds.go b/google/creds.go index cd0b03e..e215b9d 100644 --- a/google/creds.go +++ b/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) } diff --git a/google/google.go b/google/google.go index 9fa76cf..38a3677 100644 --- a/google/google.go +++ b/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...) } diff --git a/main.go b/main.go index 0dc2063..16d7606 100644 --- a/main.go +++ b/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" diff --git a/slack/config.go b/slack/config.go index 9384c7f..0855593 100644 --- a/slack/config.go +++ b/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...) } diff --git a/zoom/creds.go b/zoom/creds.go index 0493094..64ee8b9 100644 --- a/zoom/creds.go +++ b/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) } diff --git a/zoom/zoom.go b/zoom/zoom.go index 6c387f5..e4236db 100644 --- a/zoom/zoom.go +++ b/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 {