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

feat: add anvil_mine_detailed #460

Merged

Conversation

itegulov
Copy link
Contributor

@itegulov itegulov commented Dec 2, 2024

What 💻

Closes #454

The RPC is fairly straightforward but testing it required me to go on a rabbit hole that uncovered that alloy-zksync can't query transactions or hydrated blocks right now. popzxc/alloy-zksync#29 is what I arrived at while working on this, e2e-tests-rust has been updated to use that branch.

Why ✋

Feature parity with anvil

@itegulov itegulov requested a review from a team as a code owner December 2, 2024 08:10
@dutterbutter dutterbutter merged commit 928fea4 into daniyar/manual-mine-non-empty-blocks Dec 2, 2024
3 checks passed
@dutterbutter dutterbutter deleted the daniyar/anvil/mine-detailed branch December 2, 2024 13:55
itegulov added a commit that referenced this pull request Dec 3, 2024
* slightly reduce lock-related boilerplate

* make sure system cotnracts are immutable

* move `SystemContracts` to base node to minimize locking

* make manual mining take txs from mempool

* feat: add `anvil_mine_detailed` (#460)

* fix `v` calculation for legacy txs

* add `anvil_mine_detailed`
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