-
Notifications
You must be signed in to change notification settings - Fork 378
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 a whole bunch of deprecation warnings #6887
Conversation
fa7ec01
to
f03aed0
Compare
f03aed0
to
309a096
Compare
The broken tests look like they are caused by upgrading semantic-version. |
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.
Looks good if you get the semver problems sorted out.
@@ -12,6 +12,7 @@ Flask-Cors | |||
Flask-RESTful | |||
gevent>=1.5a3 | |||
guppy3 | |||
jinja2>=2.11 |
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.
I'm not sure why we need this explicitly.
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.
Because versions before that used old deprecated collections
imports (ABC base classes) which will break starting with Python 3.9.
We had accrued a big bunch of deprecation warnings (which usually showed up when running the tests) which would cause breakage in future Python versions. This upgrades packages as necessary and fixes most warnings. The remaining ones are being tracked here: - raiden-network/raiden-api-client#1 - matrix-org/synapse#9641 - matrix-org/synapse#9642 - slezica/python-frozendict#25 This also temporarily removes the docs requirements from the dev set since there's a conflict between `releases` and `py-solc-x` pinned versions of `semantic-version`. See: bitprophet/releases#94
309a096
to
8af9854
Compare
🎉 Great news! Looks like all the dependencies have been resolved: 💡 To add or remove a dependency please update this issue/PR description. Brought to you by Dependent Issues (:robot: ). Happy coding! |
Description
We had accrued a big bunch of deprecation warnings (which usually showed up when running the tests) which would cause reakage in future Python versions.
This upgrades packages as necessary and fixes most warnings.
The remaining ones are being tracked here:
This also temporarily removes the docs requirements from the dev set since there's a conflict between
releases
andpy-solc-x
pinned versions ofsemantic-version
. See: bitprophet/releases#94This requires some fixes in raiden-contracts.
Requires raiden-network/raiden-contracts#1443