PR: Add Python 3.9 compatibility for `collections.abc` module by tirkarthi · Pull Request #204 · spyder-ide/qtpy · GitHub
Skip to content

PR: Add Python 3.9 compatibility for collections.abc module - #204

Merged
dalthviz merged 4 commits into
spyder-ide:masterfrom
tirkarthi:fix-abc-warnings
Aug 10, 2021
Merged

dalthviz merged 4 commits into
spyder-ide:masterfrom
tirkarthi:fix-abc-warnings

Conversation

@tirkarthi

@tirkarthi tirkarthi commented Dec 7, 2019

Copy link
Copy Markdown
Contributor

@tirkarthi

Copy link
Copy Markdown
Contributor Author

@tirkarthi tirkarthi changed the title Import ABC from collections.abc for Python 3.9 compatibility. Add Python 3.9 compatibility. Mar 10, 2020
Comment thread qtpy/uic.py Outdated
@graingert

Copy link
Copy Markdown

@goanpeca it might be nicer to use from six.moves.collections_abc import Callable

it seems that it would be more idiomatic to use six everywhere in place of qtpy.py3compat

@CAM-Gerlach CAM-Gerlach changed the title Add Python 3.9 compatibility. Add Python 3.9 compatibility Mar 19, 2021
@dalthviz dalthviz changed the title Add Python 3.9 compatibility PR: Add Python 3.9 compatibility Aug 7, 2021
@dalthviz dalthviz closed this Aug 7, 2021
@dalthviz dalthviz reopened this Aug 7, 2021
Comment thread qtpy/uic.py Outdated
Co-authored-by: Daniel Althviz Moré <d.althviz10@uniandes.edu.co>
@dalthviz

dalthviz commented Aug 10, 2021

Copy link
Copy Markdown
Member

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 :)

@dalthviz dalthviz changed the title PR: Add Python 3.9 compatibility PR: Add Python 3.9 compatibility for collections.abc module Aug 10, 2021
@dalthviz dalthviz added this to the v1.10.0 milestone Aug 10, 2021
@dalthviz
dalthviz requested a review from ccordoba12 August 10, 2021 08:18

@dalthviz dalthviz left a comment

Copy link
Copy Markdown
Member

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! 👍

@dalthviz
dalthviz merged commit 508f9aa into spyder-ide:master Aug 10, 2021
@tirkarthi

Copy link
Copy Markdown
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants