-
Notifications
You must be signed in to change notification settings - Fork 91
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
Add yaml/yml formatting support #540
Comments
I really worry about this plugin getting too bloated. More languages means more dependencies to download, even if users don't have any need for that part of this plugin. I think long-term, this plugin needs to either:
In the near-term, if there's an easy way to add yaml support and somebody is willing to do that work, I say go ahead. It will just be more features I won't use. 😺 |
@hazendaz |
I used a test file from https://stackoverflow.com/questions/24418449/pretty-output-with-pyyaml
and jackson would always print
not beautiful at all. |
I strongly prefer 1 than 2. It will be hard to manage the codes if you choose way 2, really hard. |
We could benefit from adding yaml support. I believe Jackson can do this but have not explored that. I did look at and try to code a working example with snakeyaml. It can at least take an extremely poorly formatted file and straighten it up but their pretty print does practically nothing to fix spacing. As yaml files are highly sensitative to spacing, if we do this we may have to get some work on upstream snakeyaml since it has pretty print option and possibly use our newer trailing space removal. There are some tools in the IDEs that do address formatting so there is possibly some adaptation of some item used in say eclipse that might work out better. Open to ideas and see some real value in formatting those files as well.
The text was updated successfully, but these errors were encountered: