golangci-lint
golangci-lint copied to clipboard
feat: Include custom linters in `enable-all`
Related to https://github.com/golangci/golangci-lint/issues/3080
.golangci.yml
linters-settings:
custom:
example:
path: ./example.so
linters:
enable-all: true
Custom linters work for enable-all
.
Hey, thank you for opening your first Pull Request !
Hello,
for security reasons, I think it's a bad idea to do that.
Hello, Thanks for review!
What security reasons do you have?
I thought about the subject and I think there is a bug because the activation of plugins should be done by default. https://github.com/golangci/golangci-lint/blob/011252127f53cf6a0f2b28dc97fed55508575b09/pkg/lint/lintersdb/manager.go#L983
The current implementation inside golangci-lint has some design problems.
I think I will fix design problems and the behavior of the plugin in another PR.
@ldez hello idez !
The current implementation inside golangci-lint has some design problems. I think I will fix design problems and the behavior of the plugin in another PR.
If you respond with another PR, Should this PR be closed ?
Yes, I will create a new PR tomorrow.