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

Adding Python3.9 support #1774

Merged
merged 1 commit into from
Apr 15, 2021
Merged

Adding Python3.9 support #1774

merged 1 commit into from
Apr 15, 2021

Conversation

gabriellm1
Copy link
Contributor

@gabriellm1 gabriellm1 commented Oct 14, 2020

What was wrong?

web3.py doesn't support python3.9

Related to Issue #1769

How was it fixed?

Followed PR #1471 template that added support to python3.8

Todo:

Cute Animal Picture

image

@gabriellm1
Copy link
Contributor Author

Hi, trying here to make my first contribution to web3.py but can't really figure out what is happening at Circle CI checks. Any tip on what I'm able to do from here?

@kclowes
Copy link
Collaborator

kclowes commented Oct 14, 2020

Awesome, thanks @gabriellm1! What happens if you remove the --no-use-pep517 flag from tox.ini?

@gabriellm1
Copy link
Contributor Author

That doesn't appear to change anything

@kclowes
Copy link
Collaborator

kclowes commented Oct 14, 2020

hmm, bummer. I can take a look. Maybe there's a dependency or two that needs updating or something 🤔

@pipermerriam
Copy link
Member

cc @davesque looks like you're the man behind the curtain for the blake2b-py library. I'm curious if you could take a look at the error from this build: https://app.circleci.com/pipelines/github/ethereum/web3.py/515/workflows/13bd2625-ad93-452f-8228-73e7f82a0e5f/jobs/129045

Looks like maybe your library needs a new build published to support python 3.9?

@gabriellm1
Copy link
Contributor Author

Should I check if there is any other dependencies which doesn't support python3.9 yet? And I also commented 'typing-extensions' dependencie since there was a comment asking to do so after python3.8, have I done right?

Last, if it's not a problem would u guys please tag this PR with hacktoberfest-accepted ?

@pipermerriam
Copy link
Member

I think that before this is mergable we're going to need to officially get 3.9 supported up the dependency chain. That means this sort of update is going to need to be applied to at least all of the libraries we maintain in the dependency chain. eth-hash, eth-utils, eth-typing, eth-account, eth-tester, py-evm, py-trie, py-rlp, eth-rlp (there might be more...)

@kclowes
Copy link
Collaborator

kclowes commented Oct 15, 2020

Sorry @gabriellm1, I was hoping this would be a quick update! Thanks for the work!

@gabriellm1
Copy link
Contributor Author

No problem guys, happy to help. Feel free to contact me if there is any update on this.

@kclowes kclowes mentioned this pull request Oct 30, 2020
@fubuloubu
Copy link
Contributor

blake2b-py was released with 3.9 support:
ethereum/blake2b-py#3 (comment)

Copy link
Member

@wolovim wolovim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it lives! 🚀

@kclowes kclowes merged commit bd6d24c into ethereum:master Apr 15, 2021
@wolovim wolovim mentioned this pull request Apr 28, 2021
@k128 k128 mentioned this pull request Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants