You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
mattsse opened this issue
Sep 5, 2024
· 5 comments
· Fixed by #11230
Assignees
Labels
A-op-rethRelated to Optimism and op-rethC-debtRefactor of code section that is hard to understand or maintainD-complexQuite challenging from either a design or technical perspective Ask for help!
emhane
added
C-debt
Refactor of code section that is hard to understand or maintain
D-complex
Quite challenging from either a design or technical perspective Ask for help!
A-op-reth
Related to Optimism and op-reth
and removed
C-enhancement
New feature or request
D-good-first-issue
Nice and easy! A great choice to get started
labels
Sep 18, 2024
A-op-rethRelated to Optimism and op-rethC-debtRefactor of code section that is hard to understand or maintainD-complexQuite challenging from either a design or technical perspective Ask for help!
Describe the feature
with #10520 we moved some optimism receipt root functions to optimism consensus crate
we still have some in primitives that should be moved:
reth/crates/primitives/src/proofs.rs
Lines 90 to 91 in 98b214f
this has some side-effects on other code, mainly:
reth/crates/primitives/src/receipt.rs
Lines 112 to 114 in 98b214f
reth/crates/evm/execution-types/src/execution_outcome.rs
Lines 207 to 208 in 98b214f
those should be moved as well, and perhaps converted to a trait
Additional context
No response
The text was updated successfully, but these errors were encountered: