-
Notifications
You must be signed in to change notification settings - Fork 864
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
web3 upgrade #339
web3 upgrade #339
Conversation
removed python 3.6
I see some more failing test so this is WIP still. I will look at this tomorrow some. |
I think I cleared up most of the test failures. All are passing locally except |
@dbfreem awesome, thanks for the PR. I can see tests for Python 3.7 still fail in Travis https://app.travis-ci.com/github/blockchain-etl/ethereum-etl/jobs/567463269
Any idea how to fix it? |
So there is a current issue where web3.py really only supports python 3.7.2 and higher so I think we need to set that in setup.py. Instead of setting it to 3.7 Do you have a link to where you saw this in travis so I can verify it is fixed? I will take a look at this tomorrow. |
I suspect because we're dropping python 3.6 support, I'll need to bump the major version? Not sure what's the best practice here |
@dbfreem re |
@medvedev1088 take a look at this now. This should be good to go now. Not sure why I didn't see the Travis integration before on the PR but they all passed now. |
Thanks @dbfreem! Will review and test it this week |
Hello, I understood blockchain-etl does not work on Python 3.10 and this PR could make it compatible again. It's because the old web3 transitively depends on an ancient version of eth-account, which does not have this tiny 4 letters fix merged and apparently Python 3.10 drop support for the old & deprecated collection API. |
Looking deeper at web3.py support for 3.10 it looks like it is officially only supported in the v6 version that is in beta now. Once that moves to an official release ethereum-etl could be upgrade to v6 and at least from a web3.py standpoint be supported on 3.10. I can't speak for other dependencies on ethereum-etl that may or may not support 3.10. I can create an issue in ethereum-etl to track this for when web3.py releases it's v6 |
Here are the changes to upgrade web3.py for issue #308.
Let me know what you think, happy to tweak this some more if needed.