-
Notifications
You must be signed in to change notification settings - Fork 214
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
Validate empty package before analysing #267
Conversation
Welcome @daemon1024! |
From #220 (comment)
The change in this PR is more in line with what's mentioned in the above comment. The caveat here is that we will be loading packages twice. Here's the commit daemon1024/klog@71d4862 using file walk. Ref #214 (comment) |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: daemon1024 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 |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
I have a change ready that integrates logcheck into golangci-lint. Then golangci-lint covers package discovery and this change here is no longer needed. /hold |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
@daemon1024: 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. |
/close This will be solved via integration into golangci-lint: kubernetes/kubernetes#108159 |
@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. |
What this PR does / why we need it:
Early return in case no packages in the provided directory, else execute the analyzer as usual
Which issue(s) this PR fixes
Fixes #214
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
Release note: