Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactoring: Fully Migrate to Alloy #48

Open
wants to merge 12 commits into
base: 47-migrate-to-alloy-rs
Choose a base branch
from

Conversation

Uniblake
Copy link

Migrate from ethers to alloy

Please carefully review:

  • abstract_wallet was rewritten due to the different trait in Alloy, not 100% sure what I did is correct.
  • parse.rs has been adapted for the alloy errors.
  • utils/pretty.rs has been heavily changed for event formatting, also please check the src/dvf.rslog parsing changes together.
  • Due to some quick-node rpc problems, customized types were introduced in utils/web3.rs for desired deserde. This may not work on another rpc?
  • The hex str representation of U256 in test_snapshot_get was changed.

@Uniblake
Copy link
Author

Added support of #46
Note, the validation of snapshot were only enforced if it is constructed with snapshot_from_diff_traces or snapshot_from_tx_ids. Snapshot retrieved with debug_storageRangeAt is not validated.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants