Repository navigation
OpenAPI: use yaml linter - #14686
OpenAPI: use yaml linter #14686
Conversation
3185c13 to
f272c5c
Compare
Fokko
left a comment
There was a problem hiding this comment.
This looks great @kevinjqliu, one comment around the line length
|
|
||
| rules: | ||
| line-length: | ||
| max: 350 |
There was a problem hiding this comment.
yea, agreed. im deferring the line-length lint fix for later. want to get the linter installed asap
geruh
left a comment
There was a problem hiding this comment.
Nice LGTM! a yaml linter that plays well with the headers
|
All changes to Specifically, here are the list of yamllint errors:
Ignored
|
|
sent out an email to dev@. if there are no objections by EOW, i'll merge this PR https://lists.apache.org/thread/g4yqzltzs2n4jsdpln66yzz439vy0bpy |
|
thanks for the review everyone! |
This PR adds a yaml linter for
rest-catalog-open-api.yamland../aws/src/main/resources/s3-signer-open-api.yamlto keep formatting consistent, usingyamllintto check for formatting issues.Introduced a few new Makefile commands
The linter rules are kept in
open-api/.yamllint