search

georust / proj

Rust bindings for the latest stable release of PROJ
https://docs.rs/proj
Apache License 2.0
137 stars 45 forks source link

Update `proj::Proj` constructors to return `Result` instead of `Option`. #98

Closed frewsxcv closed 2 years ago

frewsxcv commented 2 years ago

Surface the underlying error from PROJ.

frewsxcv commented 2 years ago

Test failure look unrelated?

michaelkirk commented 2 years ago

Test failure look unrelated?

Indeed! I'm looking into that.

michaelkirk commented 2 years ago

Test failure look unrelated?

Indeed! I'm looking into that.

I feel pretty confident that the current output of CI is failing not due to your changes. Unfortunately, I don't have any leads on fixing the problem yet. 😓

You can look at #99 to see where I'm gathering data about the issue.

frewsxcv commented 2 years ago

Squashed

bors r=michaelkirk

bors[bot] commented 2 years ago

Build failed: