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 a whole bunch of deprecation warnings #6887

Merged
merged 1 commit into from
Mar 23, 2021

Conversation

ulope
Copy link
Collaborator

@ulope ulope commented Mar 18, 2021

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 and py-solc-x pinned versions of semantic-version. See: bitprophet/releases#94

This requires some fixes in raiden-contracts.
Requires raiden-network/raiden-contracts#1443

@auto-assign auto-assign bot requested a review from karlb March 18, 2021 11:03
@ulope ulope force-pushed the fix/deprecations-2021.03 branch from fa7ec01 to f03aed0 Compare March 18, 2021 11:05
@ulope ulope force-pushed the fix/deprecations-2021.03 branch from f03aed0 to 309a096 Compare March 18, 2021 11:12
@karlb
Copy link
Contributor

karlb commented Mar 18, 2021

The broken tests look like they are caused by upgrading semantic-version.

Copy link
Contributor

@karlb karlb left a 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
Copy link
Contributor

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.

Copy link
Collaborator Author

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.

@github-actions github-actions bot added the State / Blocked Issue is blocked by another. Automatically managed. label Mar 19, 2021
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
@ulope ulope force-pushed the fix/deprecations-2021.03 branch from 309a096 to 8af9854 Compare March 23, 2021 13:50
@github-actions github-actions bot removed the State / Blocked Issue is blocked by another. Automatically managed. label Mar 23, 2021
@github-actions
Copy link

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

@ulope ulope merged commit a7a401a into raiden-network:develop Mar 23, 2021
@ulope ulope deleted the fix/deprecations-2021.03 branch March 23, 2021 14:18
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.

2 participants