8000 chore: Upgrade golangci-lint to v2 by kakkoyun · Pull Request #779 · prometheus/common · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: Upgrade golangci-lint to v2 #779

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 1 commit into from
Apr 8, 2025
Merged

Conversation

kakkoyun
Copy link
Member
@kakkoyun kakkoyun commented Apr 3, 2025
  • Migrate the configuration file
  • Update the tooling version and CI
  • Apply auto-fixes

Supersedes #778
xref: prometheus/prometheus#16356

Signed-off-by: Kemal Akkoyun kemal.akkoyun@datadoghq.com

@kakkoyun kakkoyun requested a review from SuperQ April 3, 2025 12:15
@kakkoyun kakkoyun force-pushed the migrate_to_golangci_lint_v2 branch from c991ee6 to 994c4f0 Compare April 3, 2025 12:16
@kakkoyun kakkoyun force-pushed the migrate_to_golangci_lint_v2 branch from 994c4f0 to 7af5245 Compare April 3, 2025 13:59
- Migrate the configuration file
- Update the tooling version and CI
- Apply auto-fixes

Signed-off-by: Kemal Akkoyun <kemal.akkoyun@datadoghq.com>
@kakkoyun kakkoyun force-pushed the migrate_to_golangci_lint_v2 branch from 7af5245 to b6f046b Compare April 7, 2025 15:20
@kakkoyun kakkoyun requested a review from SuperQ April 7, 2025 15:20
Copy link
Member
@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Thanks!

@SuperQ SuperQ merged commit cffef2d into main Apr 8, 2025
9 checks passed
@SuperQ SuperQ deleted the migrate_to_golangci_lint_v2 branch April 8, 2025 09:10
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.

2 participants
0