search

georust / geos

Rust bindings for GEOS
https://docs.rs/geos/
MIT License
122 stars 42 forks source link

Return a GeosResult on the various binary predicate methods #11

Closed mthh closed 6 years ago

mthh commented 6 years ago

This PR wraps the result of the various binary predicates on GGeom and PreparedGGeom in a GeosResult in order to fix #10 .

I wanted to include a test case but I can't figure out how to trigger such an exception for now.

It bumps the version number to 1.1.0 as it breaks the API.

mthh commented 6 years ago

@antoine-de Sorry, I planned to publish this new release on crates.io but I do not seem to have the necessary rights. Any advice ?

antoine-de commented 6 years ago

I just added the all of the GeoRust/core team to the crates.io owner, should work now, sorry :confused:

However I published the crates.io yesterday before reading the crates.io documentation, so we need to wait for another PR to test if you can publish a version.

btw not related at all, but we'll go to the state of the map france this year (to explain why we needed rust-geos :stuck_out_tongue_winking_eye: ), will you go there to ?

mthh commented 6 years ago

Oh great for SOTM-fr, I will be there too! (only from Friday night I think, because of other commitments + I didn't submit anything this year). For sure I will not miss your intervention if it is Saturday (otherwise let's just grab a coffee there!)