Skip to content

chore: enable staticcheck #1842

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
May 5, 2025
Merged

Conversation

mmorel-35
Copy link
Contributor

@mmorel-35 mmorel-35 commented Apr 19, 2025

Description

  • Adding staticcheck to the list of enabled linters in the Golang CI configuration
  • It excludes all the rules with which the code doesn't comply yet and will be fixed in dedicated PR and fixed the quick fixes.

@mmorel-35 mmorel-35 force-pushed the golangci-lint/staticcheck branch from d149eca to e2d50d9 Compare April 25, 2025 20:52
@mmorel-35 mmorel-35 force-pushed the golangci-lint/staticcheck branch 4 times, most recently from e510e1a to fa5a2de Compare April 25, 2025 21:06
@mmorel-35 mmorel-35 force-pushed the golangci-lint/staticcheck branch 2 times, most recently from 7a7247a to 742a7d8 Compare April 25, 2025 21:15
@mmorel-35 mmorel-35 force-pushed the golangci-lint/staticcheck branch 3 times, most recently from 19af8b5 to fa54bee Compare April 29, 2025 05:19
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
@mmorel-35 mmorel-35 force-pushed the golangci-lint/staticcheck branch from fa54bee to cefed48 Compare April 30, 2025 13:41
@mmorel-35 mmorel-35 requested review from shirou and ccoVeille April 30, 2025 16:07
Copy link
Owner

@shirou shirou left a comment

Choose a reason for hiding this comment

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

Thank you so much!

@shirou shirou merged commit ec85c0b into shirou:master May 5, 2025
52 checks passed
@mmorel-35 mmorel-35 deleted the golangci-lint/staticcheck branch May 5, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants