-
Notifications
You must be signed in to change notification settings - Fork 47
New linter yamlfmt #1011
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
New linter yamlfmt #1011
Conversation
⏱️ 13m total CI duration on this PR
|
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 for the contribution! I've left a few notes.
@det Do we miss anything else? |
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 for the contribution
@EliSchleifer @det Is it ready for merge? |
@dex4er - thanks for the contribution. will get a release out soon with this included. |
This is a configuration for https://github.com/google/yamlfmt which is the fast YAML formatter that works with different settings, ie. can format arrays with Kubernetes-style (without indentation).