Skip to content
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

Fix docs #70

Merged
merged 3 commits into from
Nov 26, 2024
Merged

Fix docs #70

merged 3 commits into from
Nov 26, 2024

Conversation

nmandery
Copy link
Owner

No description provided.

@nmandery
Copy link
Owner Author

nmandery commented Nov 25, 2024

The reason for the failing CI is that h3ronpy references the geoarrow crate by commit-id, geoarrow references the wkt crate by branch name (https://github.com/georust/wkt/tree/kyle/geo-traits-writer). That branch had some changes in the last days upgrading geo-traits vom 0.1 to 0.2. This seems to be the issue here.

It seems @kylebarron is working on this currently, so hopefully we can resolve it soon.

@kylebarron
Copy link
Contributor

Oops, apologies! this should be a lesson to always pin your git dependencies by hash 😅

Hopefully wkt and wkb will get released versions soon too

@kylebarron kylebarron mentioned this pull request Nov 25, 2024
@nmandery nmandery merged commit c6472d6 into main Nov 26, 2024
8 checks passed
@nmandery nmandery deleted the fix-docs branch December 22, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants