-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
chore(rpc): pending block helper trait #8640
Conversation
blocked by #8639 |
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.
thanks for clarifying,
lgtm
crates/rpc/rpc/src/eth/api/block.rs
Outdated
@@ -18,6 +18,8 @@ use reth_transaction_pool::TransactionPool; | |||
|
|||
use crate::eth::api::EthBlocks; | |||
|
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.
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.
see #8639 (comment)
@@ -15,8 +15,11 @@ use reth_rpc_types::FeeHistory; | |||
use reth_transaction_pool::TransactionPool; | |||
use tracing::debug; | |||
|
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.
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.
see #8639 (comment)
… emhane/ethapi-txns
Moves
EthApi
methods for loading pending block to default trait methods of new traitLoadPendingBlock
. Needed to impl default trait methods forEthBlocks
andEthTransactions
.