Skip to content

Commit e403e0d

Browse files
committed
fix clippy and docs
1 parent 1f2fca9 commit e403e0d

4 files changed

Lines changed: 12 additions & 6 deletions

File tree

.github/workflows/integration.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ jobs:
3030
- uses: Swatinem/rust-cache@v2
3131
with:
3232
cache-on-failure: true
33-
- name: Run tests
33+
- name: Run tests (unit/integration only)
3434
run: |
3535
cargo nextest run \
3636
--locked \
3737
--workspace \
38-
-E 'kind(test)' \
38+
-E 'kind(test) - package(ev-tests)' \
3939
--no-tests=warn \
4040
--no-capture
4141
@@ -49,4 +49,4 @@ jobs:
4949
- name: Decide whether the needed jobs succeeded or failed
5050
uses: re-actors/alls-green@v1.2.2
5151
with:
52-
jobs: ${{ toJSON(needs) }}
52+
jobs: ${{ toJSON(needs) }}

crates/node/src/evm_executor.rs

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,17 @@ use reth_revm::{
2626
revm::{context_interface::result::ResultAndState, database::State, DatabaseCommit, Inspector},
2727
};
2828

29-
// Local copy mirroring alloy_evm's EthTxResult for our custom executor
29+
/// Execution result wrapper used by the EV block executor.
30+
///
31+
/// This mirrors the public surface needed from alloy-evm's internal result type and
32+
/// implements `TxResult` so it integrates with the generic `BlockExecutor` trait.
3033
#[derive(Debug)]
3134
pub struct EvTxResult<H, T> {
35+
/// Result of EVM execution alongside the resulting state changes.
3236
pub result: ResultAndState<H>,
37+
/// Amount of blob gas used by the executed transaction (0 before Cancun).
3338
pub blob_gas_used: u64,
39+
/// Consensus-layer transaction type used for receipt construction.
3440
pub tx_type: T,
3541
}
3642

crates/node/src/payload_types.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@ impl TryFrom<EvBuiltPayload> for ExecutionPayloadEnvelopeV6 {
234234
block_access_list: Bytes::new(),
235235
};
236236

237-
Ok(ExecutionPayloadEnvelopeV6 {
237+
Ok(Self {
238238
execution_payload,
239239
block_value: fees,
240240
should_override_builder: false,

crates/node/src/txpool.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -593,7 +593,7 @@ mod tests {
593593
let provider = MockEthProvider::default().with_genesis_block();
594594
let evm = crate::executor::EvolveEvmConfig::new(provider.chain_spec());
595595
let blob_store = InMemoryBlobStore::default();
596-
let inner = EthTransactionValidatorBuilder::new(provider, evm.clone())
596+
let inner = EthTransactionValidatorBuilder::new(provider, evm)
597597
.no_shanghai()
598598
.no_cancun()
599599
.build(blob_store);

0 commit comments

Comments
 (0)