Skip to content

Update golangci-lint to v2 #159

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 10, 2025
Merged

Update golangci-lint to v2 #159

merged 3 commits into from
Apr 10, 2025

Conversation

oschwald
Copy link
Member

  • Bump golangci/golangci-lint-action from 6 to 7
  • Update golangci-lint config
  • fixup! Update golangci-lint config
  • Run golangci-lint fmt

Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 6 to 7.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v6...v7)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@horgh
Copy link
Contributor

horgh commented Apr 10, 2025

There is a fixup commit. I'm not sure if you want to rebase that away!

@oschwald
Copy link
Member Author

Woops, thanks! I'll rebase.

@oschwald
Copy link
Member Author

Done.

@horgh
Copy link
Contributor

horgh commented Apr 10, 2025

Also should the golangci-lint action be updated?

@oschwald
Copy link
Member Author

Also should the golangci-lint action be updated?

I believe it is.

@horgh
Copy link
Contributor

horgh commented Apr 10, 2025

Oh I must have got confused with one of the internal PRs.

@horgh horgh merged commit ff237b9 into main Apr 10, 2025
9 checks passed
@horgh horgh deleted the greg/eng-1764 branch April 10, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants