-
Notifications
You must be signed in to change notification settings - Fork 188
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
Remove explicit nil
error initialization.
#114
Conversation
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@jsafrane initialization of |
Instead of explaining in a PR comment why this change is useful, better put that into the commit message. Then it'll show up automatically in the PR once you submit it and also will get recorded in the git history. Having said that, the change is good. Thanks for taking the time to clean this up. /lgtm |
@pohly sure, will keep it in mind. Also updated in PR subject, so that it will be there in the commit history. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: humblec, jsafrane The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
initialization of nil make go-lint unhappy. Removing it via this PR.
Signed-off-by: Humble Chirammal hchiramm@redhat.com