Skip to content

Conversation

@stevehipwell
Copy link
Collaborator

Resolves #2861


Before the change?

  • Uses golangci-lint v1

After the change?

  • Uses golangci-lint v1
  • Improved linting

Pull request checklist

  • Schema migrations have been created if needed (example)
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@stevehipwell stevehipwell force-pushed the golangci-lint-v2 branch 2 times, most recently from c7811ed to f2f3171 Compare November 14, 2025 11:26
Signed-off-by: Steve Hipwell <steve.hipwell@gmail.com>
@stevehipwell stevehipwell marked this pull request as ready for review November 14, 2025 13:17
@stevehipwell
Copy link
Collaborator Author

@nickfloyd could you please review this PR?

Copy link
Member

@nickfloyd nickfloyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will definitely make #2891 a bit of a mess, but this is worth it. Thanks for getting the knocked out! ❤️

@stevehipwell stevehipwell merged commit b691d8f into integrations:main Nov 14, 2025
7 checks passed
@stevehipwell stevehipwell deleted the golangci-lint-v2 branch November 14, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MAINT]: Upgrade golangci-lint to `v2

2 participants