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

Update minimist to latest #13

Closed hkuchibhotla closed 4 years ago

hkuchibhotla commented 4 years ago

This repo uses minmist 0.0.7, which seems to have some security vulnerabilities. Can we update to the latest 1.2.5

   "minimist": "1.2.5"

Tests Passed

ungoldman commented 4 years ago

fine by me, PR welcome

hkuchibhotla commented 4 years ago

@ungoldman Thank you, can i get write access to commit/ open a PR?

ungoldman commented 4 years ago

@hkuchibhotla you can fork this repo and open a PR by following these instructions: https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request-from-a-fork

hkuchibhotla commented 4 years ago

Thank you @ungoldman, here it is: https://github.com/maxogden/simplify-geojson/pull/14

ungoldman commented 4 years ago

Thank you @hkuchibhotla! Resolved in #14, published in v1.0.4