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
refactor: Drop reth_primitives::Log #7651
refactor: Drop reth_primitives::Log #7651
Conversation
ad5bad5
to
e206cd8
Compare
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.
this is great!
a few nits
@mattsse I need help for the failed test at line 540 file proofs.rs. The input is identical, but the output seems to be difference cmpared to primitive::Log. |
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.
looks like something went wrong while migrating byte data for one test.
I quickly redid it and test passes
tysm!
Drop
reth_primitives::Log
since we have implemented compact_codec for it instorage/codecs
crate.Ref #7050
WIP