Feature request: Native comments-preserving conversion from YAML -> JSONNET

[ghost]

In the evolution of piles of YAML to JSONNET, we find ourselves converting YAML -> JSON -> JSONNET a lot. When we do this, YAML comments are dropped in the lossy conversion from YAML -> JSON. However, if the conversion were from YAML -> JSONNET that conversion needn't and shouldn't be lossy with respect to comments.

Thoughts?

Thanks!

[sparkprime]

Know of a YAML parser that preserves whitespace and comments? YAML is a very complex format so writing a parser is hard.

…

On Tue, 13 Aug 2019 at 18:40, Joshua Dotson ***@***.***> wrote: In the evolution of piles of YAML to JSONNET, we find ourselves converting YAML -> JSON -> JSONNET a *lot*. When we do this, YAML comments are dropped in the lossy conversion from YAML -> JSON. However, if the conversion were from YAML -> JSONNET that conversion needn't and shouldn't be lossy with respect to comments. Thoughts? Thanks! — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#698?email_source=notifications&email_token=AABJBXRJ5P2X55WSGYKHTKTQEMZ5NA5CNFSM4ILPMKC2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFCLCTA>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABJBXRB5YXRHVG5QVFVWI3QEMZ5NANCNFSM4ILPMKCQ> .

[sbarzowski]

This could be pretty useful. I think it should be a completely separate tool, unless there is a some reason for tighter integration.

The only parser that doesn't drop comments that I could find is ruamel.yaml. However comment extraction seems to be far from trivial: https://stackoverflow.com/questions/55130078/yaml-parsing-with-comments-for-doc-purpose

[ghost]

Thanks for the replies. I looked at ruamel.yaml and https://github.com/mohsen1/yawn-yaml as well, neither of which would easily integrate with either of jsonnet's two implementations. A freestanding tool would be fine with me. Feel free to close this since standalone is the only practical way forward at this time.

[sparkprime]

Maybe it's possible to use that to make a YAML -> JSON+comments converter, which can then be input into jsonnetfmt.

[sbarzowski]

This is tracked by #765.