-
-
Notifications
You must be signed in to change notification settings - Fork 153
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
PR: Add Python 3.9 compatibility for collections.abc
module
#204
Conversation
I am not sure the build failures in some of the pipelines are related to my PR. |
@goanpeca it might be nicer to use it seems that it would be more idiomatic to use |
Co-authored-by: Daniel Althviz Moré <d.althviz10@uniandes.edu.co>
Hi @tirkarthi , after talking with @ccordoba12 I will do some changes here to wrap up the work and move forward for the release (basically removing the PY33 logic and the line for the widgets iteration to then follow up the widgets change at #224). Hope this is ok with you @tirkarthi :) |
collections.abc
module
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.
Thank you @tirkarthi for your work in this one! 👍
Thanks @dalthviz |
ImportError
in Python 3.9 and raisesDeprecationWarning
from 3.4 . Hence usecollections.abc
and also preserve compatibility with Python 2. Ref : bpo-25988: Do not expose abstract collection classes in the collections module. python/cpython#10596Uselist
instead of deprecated getchildren in Python 3. DeprecationWarning for getchildren #206 PySide2 and Python3.9: xml.etree.ElementTree.Element' object has no attribute 'getchildren #238