Skip to content

chore: enable several rules from go-critic#741

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

chore: enable several rules from go-critic#741
SuperQ merged 10 commits intoprometheus:masterfrom
mmorel-35:go-critic/enable-all

Conversation

@mmorel-35
Copy link
Contributor

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

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
Copy link
Member

@discordianfish discordianfish left a comment

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
7 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