Repository navigation
For the love of consistency... Apply linter auto-fixes - #2116
Conversation
|
Hello @JoelSpeed! Some important instructions when contributing to openshift/api: |
|
/lgtm |
98b61ad to
30be263
Compare
|
/lgtm |
0adc3cd to
00176d2
Compare
|
/retest-required |
|
@JoelSpeed: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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-sigs/prow repository. I understand the commands that are listed here. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, JoelSpeed, theobarberbany The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This PR runs
kal -fix, which auto fixes based on several linter rules.+kubebuilder:validation:Requiredis replaced with+required, all tooling now supports the latter+kubebuilder:validation:Optionalis replaced with+optional, all tooling as always supported the latterFoo -> foo)This means that anyone who now sees linter errors, has a chance to run
make lint-fixand have it fix only their issues, from their PR, rather than having it fix the who repository.I've run the generation, and checked painstakingly, that there are no functional changes to the output of the generated CRDs. All that we have updated here, are markers, and documentation, to be consistent.
I hope to merge this quickly as the merge conflicts will be hell.