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.

Sign up for GitHub

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

Jump to bottom

update rio-cogeo for newer morecantile #298

Merged
merged 4 commits into from
Nov 25, 2024
Merged

update rio-cogeo for newer morecantile #298

merged 4 commits into from
Nov 25, 2024

Conversation

AndrewAnnex
Copy link
Contributor

nothing I saw at a quick glance needed changing so let's see what happens with the ci

@AndrewAnnex
Copy link
Contributor Author

hmm okay I though the builds didn't need approval, I'll look again tomorrow and build locally/actually read the changelogs

@AndrewAnnex
Copy link
Contributor Author

needed to drop python 3.8 in this PR as rasterio isn't shipping wheels anymore for python 3.8 after version 1.4

vincentsarago
vincentsarago reviewed Oct 28, 2024
View reviewed changes
pyproject.toml Outdated Show resolved Hide resolved
vincentsarago
vincentsarago reviewed Oct 28, 2024
View reviewed changes
pyproject.toml Outdated Show resolved Hide resolved
@AndrewAnnex AndrewAnnex changed the title update rio-cogeo for newer morecantile and rasterio update rio-cogeo for newer morecantile Oct 28, 2024
@AndrewAnnex
Copy link
Contributor Author

hey @vincentsarago wondering if we can merge this as it's blocking some CI builds I have, if something needs fixing happy to do it.

@vincentsarago
Copy link
Member

Sorry I've been super busy recently with travelling/conferences
Not sure about timeline as next week looks also super busy I'm sorry

@AndrewAnnex
Copy link
Contributor Author

@vincentsarago are there more significant changes needed outside of updating the version pin? if so then that's cool and understandable, i'd take a stab at them but nothing I saw when I looked last needed updates... but maybe I am totally missing something

@vincentsarago
Copy link
Member

I do not now

@vincentsarago vincentsarago merged commit f41ea11 into cogeotiff:main Nov 25, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincentsarago vincentsarago Awaiting requested review from vincentsarago

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AndrewAnnex @vincentsarago