Fix linting - #1181
Conversation
…sh -c '! egrep --with-filename --line-number "^( |\t+ )" "$@"' succeed
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: tmstff The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Important Review skippedToo many files! This PR contains 959 files, which is 809 over the limit of 150. To get a review, narrow the scope: Upgrade to a paid plan to raise the limit. ⚙️ Run configurationConfiguration used: Repository: openshift-online/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (959)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
PR needs rebase. 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. |
Description
To get
make lintproduce no errors (on MacOs / BSD), I had to change a lot of whitespaces. Not sure if this is right, maybe instead themake lintimplementation should be updated?Merge after #1180 (as it's based on that branch)
Type of Change
make updateafter model change)Verification
make checkpasses (model syntax validation)make verifypasses (generated code matches model)make lintpasses (indentation enforcement)Checklist
clientapi/andopenapi/are updated if model changed (make update)