ropensci / geojsonio

Convert many data formats to & from GeoJSON & TopoJSON
https://docs.ropensci.org/geojsonio
Other
150 stars 59 forks source link

Remove dependency on geojsonlint? #201

Closed maelle closed 1 year ago

maelle commented 1 year ago

The call for a new maintainer hasn't been successful (yet), we might need to archive it some time after next month (I'd wait as long as necessary).

mikemahoney218 commented 1 year ago

Checking CI in #202 now, but this might be a pretty trivial fix (:crossed_fingers:)

mikemahoney218 commented 1 year ago

@maelle Looks like this will be easy to remove. Do you have a timetable on when you'd like to archive the package? I'd be happy to cut a quick patch release to remove the package, if sooner is better.

maelle commented 1 year ago

No hurry! I'm only worried someone might decide to take a dependency on it in the meantime, shifting the whole timeline. :grin:

maelle commented 1 year ago

thank you!!

mikemahoney218 commented 1 year ago

Don't thank me yet, hasn't made it to CRAN :laughing: I'm going to try to find time to put the patch out today, to make sure I don't forget. Just opened #203 to track that :smile:

maelle commented 1 year ago

I still wanted to say thank you for looking into it!

github-actions[bot] commented 1 year ago

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.