Skip to content

Conversation

@Antonboom
Copy link
Contributor

Closes #4134

@ldez, hi!

I reviewed all linters and yes, you are right:

The majority of those empty defaults are real.

If you don't see the point in this pr, you can close it, I won't mind.

Thanks

@Antonboom Antonboom requested a review from ldez October 19, 2023 18:22
@ldez ldez changed the title govet: doc: describe enabled and disabled by default analyzers docs: describe enabled and disabled by default analyzers of govet Oct 19, 2023
@ldez ldez added enhancement New feature or improvement area: docs labels Oct 19, 2023
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM

@ldez ldez merged commit 1bfcc5f into golangci:master Oct 19, 2023
@ldez ldez modified the milestone: v1.55 Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs enhancement New feature or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

doc: check default lists configuration reference

2 participants