Conversation
mihaic
left a comment
There was a problem hiding this comment.
Great idea! Can we use another repo? The pre-commit mirror repo has been archived.
|
How about using |
It looks like it is not keeping up with ruamel.yaml development:
https://github.com/jumanjihouse/pre-commit-hook-yamlfmt/blob/master/setup.py#L12 I am not convinced by ymlfmt either but I will remove my change request in case someone else wants to approve. |
ethanglaser
left a comment
There was a problem hiding this comment.
No objections from my end. Seems like an industry standard tool with fairly typical args. Thanks for unifying
|
Okay, this time I went with All yaml files still pass the check. I tested with |
I noticed inconsistent configuration of yml autoformatters, so I added a formatter to the pre-commit hooks.