search

georust / topojson

TopoJSON bindings and utilities for Rust
MIT License
10 stars 8 forks source link

Relax geojson version requirements #11

Closed pjsier closed 3 years ago

pjsier commented 3 years ago

Closes #10. I confirmed that the tests pass on all geojson versions from 0.16 up until 0.20 (since it hasn't been published yet)

urschrei commented 3 years ago

I've just published geojson 0.20.0 – We added some more granular errors and additional From / TryFrom conversions, so it's technically breaking but in practice shouldn't affect topojson: you should be able to include 0.20.0.

pjsier commented 3 years ago

Looks like the tests still pass on 0.20.0, so I've added it here, thanks for the heads up!

pjsier commented 3 years ago

Is this all set for me to merge with the approval and update? Wanted to make sure since I haven't contributed in a while

frewsxcv commented 3 years ago

if you feel comfortable doing that then go for it! it should be just a cargo publish and ideally tagging the commit in git