-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expand docs with parsing examples and corner cases, and enable conversion docs #85
Conversation
A GeoJSON Feature object MUST include a properties field. Currently, if it's set to None when initialising a Feature struct, the field won't be serialised, resulting in invalid GeoJSON.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks great! this should make it easier for users who are just interested in parsing geojson 👌
just one comment about a url in the docs, but looks good to me!
src/lib.rs
Outdated
//! | ||
//! ## Conversion to Geo objects | ||
//! | ||
//! The [try_into](file:///Users/sth/dev/rust-geojson/target/doc/geojson/conversion/index.html) trait provides *fallible* conversions from GeoJSON Value structs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[try_into](file:///Users/sth/dev/rust-geojson/target/doc/geojson/conversion/index.html)
this should probably be a relative URL path to index.html
bors r+ |
Build succeeded |
This also inserts a blank
properties
map when serialising, if the field was set toNone
. See b51f993