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

Fix Python 3.7 deprecation warnings #67

Merged

Conversation

elprans
Copy link
Contributor

@elprans elprans commented Nov 1, 2018

Importing ABCs directly from the collections module is deprecated
in Python 3.7.

Importing ABCs directly from the `collections` module is deprecated
in Python 3.7.
@lincolnq
Copy link

lincolnq commented Feb 3, 2019

It would be nice to get this merged. What is the status?

@allardhoeve
Copy link

The CI failed, but it does not seem to be caused by this PR. It would be a good idea to add Python 3.7 to the travis XML.

@syrusakbary: can you please release a version with these warnings fixed? Python 3.7 is now the default version of Python in Homebrew (MacOSX).

@olirice
Copy link

olirice commented Oct 18, 2019

Python 3.8.0 is now out. The deprecation warning has been extended to 3.9 so this doesn't block upgrades but it would still make my day if this could get cut into a new release to kill the warnings.

As commented above, the CI failure is not related to the changes

Any chance someone has spare eyes for this?

@mrdon-reciprocity
Copy link
Contributor

mrdon-reciprocity commented Nov 21, 2019

The core problem here is mypy isn't version pegged, so new versions find new problems. I've created a PR, based on this PR, to fix the mypy errors: #79

I am unclear whether the lack of a mypy version is intentional or not, so I left that alone.

@syrusakbary syrusakbary merged commit c088f7c into syrusakbary:master Dec 14, 2019
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.

None yet

6 participants