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.

Sign up for GitHub

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

Jump to bottom

WIP fix(requests): make ExecutionRequests devnet-5 compatible #297

Draft

ryanschneider wants to merge 1 commit into develop from execution-requests-cleanup

+520 −374

Contributor

ryanschneider commented Dec 18, 2024

This will require a new reth using alloy 0.8.1 or ideally 0.8.2 or whatever ends up containing alloy-rs/alloy#1807

Still very much a work-in-progress.

📝 Summary

💡 Motivation and Context

✅ I have completed the following steps:

Run make lint
Run make test
Added tests (if applicable)


          WIP fix(requests): make ExecutionRequests devnet-5 compatible

f549c20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

dvush Awaiting requested review from dvush dvush will be requested when the pull request is marked ready for review dvush is a code owner

ZanCorDX Awaiting requested review from ZanCorDX ZanCorDX will be requested when the pull request is marked ready for review ZanCorDX is a code owner

ferranbt Awaiting requested review from ferranbt ferranbt will be requested when the pull request is marked ready for review ferranbt is a code owner

liamaharon Awaiting requested review from liamaharon liamaharon will be requested when the pull request is marked ready for review liamaharon is a code owner

metachris Awaiting requested review from metachris metachris will be requested when the pull request is marked ready for review metachris is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet