-
Notifications
You must be signed in to change notification settings - Fork 196
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
Added estimateGasBundle method #102
Added estimateGasBundle method #102
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome add, just some small (opinionated) details to discuss
This is great, thanks! Fixes #100 |
@libevm sadly there's no good place for end to end tests atm, especially for the mev-callbundle branch. Right now, end to end tests live in https://github.com/flashbots/mev-geth-demo and you can look at mev-geth/.github/workflows/go.yml Line 52 in 516e2c3
I'm fine leaving the testing outside the scope of this PR, but feel free to add it if you have the time. |
#100
Adds method
eth_estimateGasBundle
that accepts similar params toeth_callBundle
, but doesn't require the bundle to be signed, e.g.:Returns the result in the format:
Manually tested with the following script, bit unsure where to add the tests for mev-geth so some guidance would be appreciated:
Response:
Please be patient, something something etc. First time writing Go and don't really know the correct conventions.