Skip to content

Conversation

mmorel-35
Copy link
Contributor

Description

Enable and fix more go-critic rules

@ccoVeille
Copy link
Contributor

ccoVeille commented May 17, 2025

Thanks for pointing out each rule that were involved when I asked for extra information

@mmorel-35 mmorel-35 force-pushed the gocritic/enable-all branch 6 times, most recently from ae18de8 to a1fb787 Compare June 1, 2025 08:26
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
@mmorel-35 mmorel-35 force-pushed the gocritic/enable-all branch from a1fb787 to 431dd63 Compare June 1, 2025 08:31
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.

err shadowing is on going on #1863, but I think this PR can be merged. Thank you for your contribution!

@shirou shirou merged commit ce7e4c8 into shirou:master Jun 7, 2025
52 checks passed
@mmorel-35 mmorel-35 deleted the gocritic/enable-all branch June 7, 2025 14:16
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