Skip to content

Upgrade golangci-lint #4306

@alarso16

Description

@alarso16

I forgot during my last review that we also need to update CONTRIBUTING.md and README.md and the docstring on go.mod and possibly our golangci. See 1ecddc7 as an example.

Originally posted by @joshua-kim in #4304 (review)

You can see the change log here: https://golangci-lint.run/docs/product/changelog/

There are new linters that maybe should be added, as well as extra parts of other linters. noctx has many changes, but #4163 is working to resolve this

Initial recommendations after looking at the change log:

  • revive's unnecessary-format and enforce-switch-style
  • New linter embeddedstructfieldcheck

Metadata

Metadata

Assignees

No one assigned

    Labels

    ciThis focuses on changes to the CI processcleanupCode quality improvementlifecycle/staleInactive for 60 days

    Type

    No type

    Projects

    Status

    Ready 🚦

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions