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

Update package to use web3.py >=6.0.0, <7 #87

Merged
merged 6 commits into from
Jul 23, 2024

Conversation

vile
Copy link
Contributor

@vile vile commented Feb 11, 2024

This PR aims to support web3.py versions >=6.0.0, <7.

According to the web3.py v6 migration guide, only a few breaking changes apply to this project:

  • Use of consistent Pythonic snake_case for method and property names
  • renaming the sha3 method to keccak

Outside of the v6 migration guide, there is one other breaking change:

  • web3.module.Module renames the web3 property to w3 for the Web3 class (async or sync)

This PR does not add async methods (see: Web3 class split into Web3 and AsyncWeb3).

In addition to the above changes, Poetry packages were locked using the latest web3.py version (6.15.1) and the .vscode/settings.json file was changed to properly format using Black on-save. /example/simple.py has also been updated to use web3.py >=6.0.0.

vile and others added 6 commits July 24, 2024 01:12
- bump web3.py version to >=6, <7

- chore: bump package version to 2.0.0

- chore: lock packages

- chore: update vscode settings.json

- update .vscode/settings.json to properly use black formatter
- use Pythonic snake_case where applicable
- `web3.module.Module uses` `w3` naming instead of `web3`(e.g. `self.web3.eth.block_number` -> `self.w3.eth.block_number`)
- sha3 method has been renamed to keccak
@odysseus0
Copy link
Collaborator

LFG. Rebase working clean af. v6 is here.

@zeroXbrock
Copy link
Member

thanks @vile! Nicely done

@zeroXbrock zeroXbrock merged commit f0062a5 into flashbots:master Jul 23, 2024
1 check passed
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.

3 participants