Fix deprecation warnings

georust / geojson

Library for serializing the GeoJSON vector GIS file format

https://crates.io/crates/geojson

Apache License 2.0

280 stars 60 forks source link

Fix deprecation warnings #242

Closed nk9 closed 6 months ago

nk9 commented 6 months ago

[x] I agree to follow the project's code of conduct.
[ ] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

When running the project from a fresh download, there are a number of compiler warnings which I have fixed.

nk9 commented 6 months ago

Coordinate to Coord was introduced in geo_types 0.7.8
points_iter() was marked as deprecated on 4 Jan 2022, with a release it looks like of 0.7.13
The two unused imports appear to have been added in this commit, and it looks like they were never actually needed.

I'm not sure what you mean about the "non version" of geo-types? I could easily have my setup wrong, not looking to create work for you. If I've overlooked a contribution guide, or left steps out, please let me know!

michaelkirk commented 6 months ago

sorry, "non" version was an autocorrect.

Can you please bump the min version of geo types in Cargo.toml to 0.7.13?

edit: fixed min version

nk9 commented 6 months ago

Sure, all done!

michaelkirk commented 6 months ago

Thanks!

georust / geojson

Fix deprecation warnings #242

[ ] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

[ ] I added an entry to `CHANGES.md` if knowledge of this change could be valuable to users.