Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

Support YAML MML files #401

Closed
pnorman opened this issue Aug 20, 2015 · 3 comments
Closed

Support YAML MML files #401

pnorman opened this issue Aug 20, 2015 · 3 comments
Milestone

Comments

@pnorman
Copy link
Contributor

pnorman commented Aug 20, 2015

JSON has a number of problems that make it unsuitable for a multi-contributor style, including code review, merge conflicts, and difficulty editing. For this reason, projects have started to move towards defining their layers in YAML and converting it to JSON, or, in the case of Kosmtik, supporting reading YAML directly.

It would be useful if the carto program could read YAML directly, eliminating the need to convert YAML to JSON in order to produce Mapnik XML.

cc @yohanboniface

@tmcw
Copy link
Contributor

tmcw commented Dec 4, 2015

I'd accept a PR for this as long as it's backwards-compatible.

@yohanboniface
Copy link
Contributor

I'd accept a PR for this

That's somewhere on my radar, but time, priorities… ;)

as long as it's backwards-compatible.

YAML being a superset of json, that should be doable, but the devil is in the piece of code that you look for last, no? :)

@pnorman
Copy link
Contributor Author

pnorman commented Dec 10, 2015

YAML being a superset of json, that should be doable, but the devil is in the piece of code that you look for last, no? :)

I was thinking we'd need to autodetect yaml vs json, but you're right, a valid json file is a valid yaml file.

@tmcw tmcw closed this as completed in #419 Feb 26, 2016
@nebulon42 nebulon42 added this to the v0.16 milestone Mar 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants