Skip to content

chore: enable several rules from go-critic - #741

Merged
SuperQ merged 10 commits into
prometheus:masterfrom
mmorel-35:go-critic/enable-all
Oct 18, 2025
Merged

SuperQ merged 10 commits into
prometheus:masterfrom
mmorel-35:go-critic/enable-all

Conversation

@mmorel-35

@mmorel-35 mmorel-35 commented Aug 2, 2025

Copy link
Copy Markdown
Contributor

Description

Enables and fixes several rules from go-critic

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
@mmorel-35
mmorel-35 force-pushed the go-critic/enable-all branch from 1b92c61 to 9807639 Compare August 2, 2025 21:09

@discordianfish discordianfish left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@SuperQ
SuperQ merged commit 38a61f6 into prometheus:master Oct 18, 2025
2 checks passed
@mmorel-35
mmorel-35 deleted the go-critic/enable-all branch October 18, 2025 08:35
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.

3 participants