mapbox / tile-cover

Generate the minimum number of tiles to cover a geojson geometry
MIT License
189 stars 40 forks source link

Output spec compliant #82

Closed maning closed 6 years ago

maning commented 6 years ago

Running my output from tile-cover to geojsonhint, I get the following error

30637:1  warning  Polygons and MultiPolygons should follow the right-hand rule
30638:1  warning  Polygons and MultiPolygons should follow the right-hand rule

I fixed it with geojson-rewind, but a spec compliant output is better.

mourner commented 6 years ago

The conversion actually happens in tilebelt (https://github.com/mapbox/tile-cover/blob/master/index.js#L25), so you should probably open an issue (or a PR?) there instead.

maning commented 6 years ago

Thanks! Opened one: https://github.com/mapbox/tilebelt/issues/35