-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CI: Unpin openpyxl #22605
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
Comments
Upstream commit just came in to patch this: https://bitbucket.org/openpyxl/openpyxl/commits/fa13fbfda13a Need to wait on next release to unpin this. |
Reported error in https://bitbucket.org/openpyxl/openpyxl/issues/1094/error-with-pandas-datetimeindex (thru being duplicate of https://bitbucket.org/openpyxl/openpyxl/issues/1093/newly-introduced-keyerror-problem-in) was fixed in 2.5.7 apparently - current version is 2.5.12. Tagging @alimcmaster1, as he asked about just that on the bitbucket tracker. |
@h-vetinari . I already largely did this #23361 the only build that's that remain pinned are azure-macOS-35, and azure-27-compat build. So I think this is perhaps fine to close unless anyone thinks there is something else we should be doing here? |
@jreback @TomAugspurger : Do we intend to unpin Otherwise, I do agree that we should close this (and mark for |
are we pinning anywhere? |
We have builds that are not pinned though, so this is probably fine to close. Or some of the ones pinned at 2.5.5 can be lifted, either works for me. |
Closing, since we have a mix of pinned & unpinned. |
Followup to #22601 and dask/dask-ml#84 (comment).
Upstream issue: https://bitbucket.org/openpyxl/openpyxl/issues/1094/error-with-pandas-datetimeindex
(one of several in fact filed for this issue)
The text was updated successfully, but these errors were encountered: