-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
Test Failure with xlrd and defusedxml #27016
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
Do you have defusedxml installed? |
not sure.. have been messing with environment to check some SQL test setup errors. I am getting this though...
will start again with a clean environment and check. |
clean environment.. still fails. defusedxml is installed.
|
Good to know. This might be related to #26657 (comment) (cc @datapythonista) and just some general buggery in xlrd (https://github.com/python-excel/xlrd/issues/315#issuecomment-493715073) that won't get fixed. I think we should just catch these in the tests |
Is this not failing in the CI because we don't have a test build with Didn't know about |
Yea that's right. Would need to be the combination of xlrd and defusedxml which I don't think appears in CI |
any chance this has resolved itself in the interim? |
This looks okay when I run this locally
So closing since I can't reproduce and lack of activity, but happy to reopen if it appears again. |
The text was updated successfully, but these errors were encountered: