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

Set minimum click to 8.1 #1272

Merged
merged 2 commits into from
Nov 20, 2023
Merged

Conversation

jacobtomlinson
Copy link
Member

Closes #1271

@jacobtomlinson jacobtomlinson added bug Something isn't working non-breaking Non-breaking change labels Nov 9, 2023
@pentschev
Copy link
Member

I think this shouldn't be necessary anymore if dask/dask#10623 gets merged, right @jacobtomlinson ?

@jacobtomlinson
Copy link
Member Author

That is true, but that relies on us moving our pin to whatever Dask release contains that change. If we've already pinned Dask for 23.12 I would be tempted to merge this in here still.

@quasiben
Copy link
Member

Let's get this merged in for 23.12

@quasiben
Copy link
Member

@jacobtomlinson can you merge with main and resolve the conflict with pyproject.toml and then we should be good

@wence-
Copy link
Contributor

wence- commented Nov 20, 2023

I fixed the merge conflict, let's see if things run

@quasiben
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit b1f13fc into rapidsai:branch-23.12 Nov 20, 2023
20 checks passed
@vyasr vyasr mentioned this pull request Nov 20, 2023
@jacobtomlinson jacobtomlinson deleted the click-8-1 branch December 4, 2023 13:14
younseojava pushed a commit to ROCm/dask-cuda-rocm that referenced this pull request Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dask cuda CLI doesn't work with click 8.0.x
4 participants