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(deps): remove the upper bounds for pyarrow #10503

Merged
merged 2 commits into from
Dec 16, 2024

Conversation

deepyaman
Copy link
Contributor

@github-actions github-actions bot added the dependencies Issues or PRs related to dependencies label Nov 16, 2024
@deepyaman deepyaman marked this pull request as ready for review November 16, 2024 23:42
@gforsyth gforsyth added this to the 10.0 milestone Nov 18, 2024
@gforsyth
Copy link
Member

I'm in favor of this. I think we should remove more than just the pyarrow upper pin, but I'll also let @cpcloud weigh in here.

@cpcloud
Copy link
Member

cpcloud commented Dec 16, 2024

Seems like it's probably fine to remove upper bounds on some of these, pyarrow is probably the most important one, and we can do the others in follow-ups as folks have time.

@cpcloud
Copy link
Member

cpcloud commented Dec 16, 2024

@deepyaman Can you update the PR and resolve the conflicts?

@cpcloud cpcloud changed the title chore(deps): remove the upper bounds for pyarrow fix(deps): remove the upper bounds for pyarrow Dec 16, 2024
@cpcloud
Copy link
Member

cpcloud commented Dec 16, 2024

Made it a fix PR so that it'll show up in the release notes.

@cpcloud cpcloud enabled auto-merge (squash) December 16, 2024 19:50
@cpcloud cpcloud merged commit 24c0aca into ibis-project:main Dec 16, 2024
82 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Issues or PRs related to dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: please consider not setting upper bounds on dependencies
3 participants