sls-packaging icon indicating copy to clipboard operation
sls-packaging copied to clipboard

manifest.yml actually contains json

Open gracew opened this issue 8 years ago • 2 comments

gracew avatar Jul 17 '17 20:07 gracew

JSON is valid YAML, so it works out.

I do think it might be nice to switch the manifest from YAML to JSON, though. It's easier to write tooling around it in Python or bash or whatever since tooling to work with JSON is way more common than YAML. People are never really going to be looking at or writing that file manually so the benefits of YAML aren't really applicable here.

sfackler avatar Jul 24 '17 04:07 sfackler

Products and tools already have to deal with yaml (e.g., for product configuration), so I think greater uniformity trumps potential easy-of-use benefits here. (Also, though I haven't used it recently, I'd be really surprised if Python didn't have reasonable yaml support. jq is an interesting one, but still could trick into reading json with a simple json-to-yaml pre-filter... )

uschi2000 avatar Jul 24 '17 13:07 uschi2000