-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bump minimum click
to >=8.1
#10623
Bump minimum click
to >=8.1
#10623
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to this. Thanks @jacobtomlinson .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jacobtomlinson! This seems totally fine to me. Could you also make similar updates to the various mindeps
conda env files and the nightly conda package here. Otherwise, this looks good to me
Also, merging main
should fix the unrelated test_quantile
failures
Done thanks @jrbourbeau |
Thanks @jrbourbeau for approval and merging! 😄 |
In a Dask subproject, I've been running into an issue with registering commands via the
dask.cli
entrypoint if that command uses aclick.Group
on versions less thanclick==8.1
.When running the
dask
CLI I see a warning likeAttributeError: 'function' object has no attribute 'command'
and the subcommand doesn't get registered.It looks like this was fixed in
click>=8.1
. How do folks feel about bumping the minimum version?