search

max-mapper / simplify-geojson

apply the ramer-douglas-peucker line simplification to geojson features or feature collections in JS or on the CLI
177 stars 17 forks source link

use standard; fix undefined var: arg #3

Closed ungoldman closed 8 years ago

ungoldman commented 8 years ago

I noticed arg is an undefined variable (cli.js#L11). I changed it to be argv._[0] as I think that's what it's supposed to be. I threw standard in there too, hope that's alright.

ungoldman commented 8 years ago

actually I'm not sure if that's right -- should argv._[0] === '-' for that case?

max-mapper commented 8 years ago

ya thats right

ungoldman commented 8 years ago

ah ok so like cat cats.geojson | simplify-geojson - | geojsonio would do the trick in a child_process.spawn env

max-mapper commented 8 years ago

yep

ungoldman commented 8 years ago

ok merging, will add tests in another PR