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

chore: bump web3 dep #636

Closed
wants to merge 1 commit into from
Closed

chore: bump web3 dep #636

wants to merge 1 commit into from

Conversation

gosuto-inzasheru
Copy link
Collaborator

this adds support for python 3.11 as per ethereum/web3.py#2699

Copy link
Member

@Tritium-VLK Tritium-VLK left a comment

Choose a reason for hiding this comment

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

I think the issue is that it changes the some functions to checksum addresses from camel case to _ and the old format is used all over. Did you test the current actions with the new requirements?

@gosuto-inzasheru
Copy link
Collaborator Author

not sure i follow your comments.

im happy to not make changes, but can you specify the python version that was used for this current config then? 3.11 fails due to the mentioned bug in web3

@gosuto-inzasheru
Copy link
Collaborator Author

ah i see now that the action scripts are running in ci, and there they run in 3.9: https://github.com/BalancerMaxis/multisig-ops/blob/main/.github/workflows/run_auth_gen.yaml

i ran them locally, but then there is no python version defined anywhere and my env defaulted to 3.11.

set my python to 3.9 locally and all works without this fix being needed

@gosuto-inzasheru gosuto-inzasheru deleted the chore/bump_web3 branch December 5, 2023 10:05
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