-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
logcheck: support new golangci-lint plugin interface #22
Conversation
The new plugin interface was added in golangci/golangci-lint#3887 to allow settings for custom plugins. This commit implements the new plugin interface with `config` value in the settings, and passes it to flag `--config`.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: linxiulei The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This issue is currently awaiting triage. If logtools contributors determine this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Welcome @linxiulei! |
I already added such support in #18. The problem with the approach in this PR is that changes in the content of the config file do not invalidate the golangci-lint cache because golangci-lint is unaware that the config changed. Do you agree that my approach is better? |
Oh, I didn't find that. I am fine to discard this one if that one is better.
Would golangci-lint invalidate the cache if the settings have changed? Would having a separate config file be preferable if the content of the config file is verbose? |
Yes.
An |
/close In favor of #18 |
@pohly: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
The new plugin interface was added in golangci/golangci-lint#3887 to allow settings for custom plugins. This commit implements the new plugin interface with
config
value in the settings, and passes it to flag--config
.