-
Notifications
You must be signed in to change notification settings - Fork 8
build: Add kube-api-linter #1209
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
base: main
Are you sure you want to change the base?
Conversation
d120726
to
677dfa2
Compare
Please consider configuring golangci-lint to run this linter by default, so that the linter runs when we are editing the files, and so that our respective IDE can organize the results (e.g. provide a list of violations that we can navigate). If we're concerned about performance, we can scope the linter to a set of directories. |
677dfa2
to
7cf3797
Compare
While I really like the idea, I'm having difficulty getting it to play ball... |
b5395d5
to
54d08f3
Compare
@dlipovetsky Done! Integrated into |
f093a17
to
90f9c35
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.
Thank you!
Validates upstream API conventions are correctly applied.
90f9c35
to
1cea89a
Compare
Validates upstream API conventions are correctly applied.