Skip to content
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

Merged
merged 4 commits into from
Aug 10, 2021
Merged

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

merged 4 commits into from
Aug 10, 2021

Conversation

tirkarthi
Copy link
Contributor

@tirkarthi tirkarthi commented Dec 7, 2019

@tirkarthi
Copy link
Contributor Author

I am not sure the build failures in some of the pipelines are related to my PR.

@tirkarthi tirkarthi changed the title Import ABC from collections.abc for Python 3.9 compatibility. Add Python 3.9 compatibility. Mar 10, 2020
@graingert
Copy link

@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
Co-authored-by: Daniel Althviz Moré <d.althviz10@uniandes.edu.co>
@dalthviz
Copy link
Member

dalthviz commented Aug 10, 2021

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
Copy link
Member

@dalthviz dalthviz left a 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! 👍

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

Thanks @dalthviz

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