-
Notifications
You must be signed in to change notification settings - Fork 21
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
Revert temporary Dask pinning in CI #39
Conversation
Codecov Report
@@ Coverage Diff @@
## master #39 +/- ##
=======================================
Coverage 94.41% 94.41%
=======================================
Files 3 3
Lines 215 215
Branches 58 58
=======================================
Hits 203 203
Misses 8 8
Partials 4 4 Continue to review full report at Codecov.
|
The latest Dask release ( |
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.
Looks good to me. Thanks for doing this @jrbourbeau
Actually, it looks as though some tests are still failing with the old error:
|
Those are due to Dask dropping support for Python 3.6 in |
Thanks for this @jrbourbeau, merging now |
Thanks @dougiesquire! |
With
dask=2021.3.1
out we can remove the temporary version pinning in CI to workaround #27. Additionally, this PR also switches to installing packages fromconda-forge
instead ofdefaults