-
Notifications
You must be signed in to change notification settings - Fork 177
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
Upgrades web3.py version in requirements #202
base: master
Are you sure you want to change the base?
Conversation
+1 |
when merge? |
+1!!! I need this to use websockets 11! |
I made a fork of this and released it on PyPI: |
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.
+1
any update on this one? Having to use the fork provided by saleh-mir to get around dependency conflicts. |
@saleh-mir Can you update Also, this package doesn't include the solidity_keccak changes 😩 |
The main motivation behind bumping the version is to allow
websockets>=10
.The v5 versions of web3 fix the
websockets
version to <10, which capped the version of thewebsockets
library that was possible to be used. However, in the latest release the cap was removed.This update allows for latest v5 (5.31.4) and v6, which would avoid dependency conflicts with
websockets
library.