Corey Farwell

Results 416 comments of Corey Farwell

Relevant discussion: https://github.com/georust/geojson/pull/245#issuecomment-2466771255

For the time being, that seems alright to me. In which case, that wouldn't require any changes to the current dimensions implementation, right?

Sounds good to me. Conceptually, would that be much different than updating it to return `Option`? I'm fine with any approach here. Just curious how you're thinking about it.

I wish we could bundle the dimension and the non-empty geometry together. Something like `Empty | (Dimension, Geometry)` rather than `(None, Empty) | (Some(Dimension), Geometry)`.

Ah darn 😞 . You're totally right

This option is already enabled