-
Notifications
You must be signed in to change notification settings - Fork 65
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
Golangci lint #774
Golangci lint #774
Conversation
4aedf72
to
9b4cfc4
Compare
7772fb7
to
f9d429c
Compare
f9d429c
to
077f877
Compare
87ad9b3
to
8c592f1
Compare
8c592f1
to
f9fe75d
Compare
/test 4.8-e2e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few things.
Spelling corrections are 👌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
f9fe75d
to
2408867
Compare
error in check dallas: unable to rebase PR 774 from redhat-openshift-ecosystem/openshift-preflight branch main |
Create a 'make lint' target, and create an action that will run golangci-lint on a PR and fail if it has any issues. Signed-off-by: Brad P. Crochet <brad@redhat.com>
2408867
to
4fcc7ac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
for the most part I think the linter adds value. I've got the two open items I've mentioned but one is non-blocking and the other I believe we already discussed. To that end, I'll consider this approved.
Signed-off-by: Brad P. Crochet <brad@redhat.com>
4fcc7ac
to
6c4b92a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Thanks @bcrochet
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: acornett21, bcrochet, komish 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 |
Implement golangci-lint and update the codebase to adhere to the linter.