-
Notifications
You must be signed in to change notification settings - Fork 27
Add flashtestation builder tx #282
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
base: main
Are you sure you want to change the base?
Conversation
46380ae
to
41c42bf
Compare
79aff67
to
0b9b3a3
Compare
41c42bf
to
01573b2
Compare
0b9b3a3
to
581ebac
Compare
01573b2
to
32146ac
Compare
581ebac
to
8768d27
Compare
32146ac
to
2499332
Compare
Co-authored-by: Solar Mithril <[email protected]>
} | ||
} | ||
}; | ||
match result { |
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.
it looks like this conversion from the evm result to TxSimulateResult
is repeated in simulate_verify_block_proof_tx
, can you separate it into another function?
} | ||
} | ||
|
||
fn check_verify_block_proof_log(&self, logs: &[Log]) -> bool { |
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.
do the iter().any()
thing and elide this function since it'll just be a short line then
π Summary
Add flashtestations builder tx and registration logic into built block
π‘ Motivation and Context
β I have completed the following steps:
make lint
make test