Skip to content

Conversation

@theochap
Copy link
Contributor

Description

Bump reth to 1.9.3

Copilot AI review requested due to automatic review settings November 18, 2025 16:44
Copilot finished reviewing on behalf of theochap November 18, 2025 16:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR bumps the reth dependency from version 1.9.2 to 1.9.3, including necessary code adaptations for API changes in the new version.

  • Updates all reth-related dependencies in Cargo.toml from 1.9.2 to 1.9.3
  • Adapts code to accommodate API changes in reth 1.9.3, specifically around EVM environment construction
  • Updates rollup-boost dependency from v0.7.9 to v0.7.10

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
Cargo.toml Updates all reth and reth-optimism dependency versions from 1.9.2 to 1.9.3, and bumps rollup-boost from v0.7.9 to v0.7.10
Cargo.lock Reflects the updated dependency versions and transitive dependencies, including op-revm (14.0.0 → 14.1.0) and revm (33.0.0 → 33.1.0)
crates/op-rbuilder/src/builders/builder_tx.rs Adapts to API changes by constructing an EvmEnv struct instead of passing cfg and block directly to try_into_tx_env; adds EvmEnv import

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@avalonche avalonche enabled auto-merge (squash) November 18, 2025 16:58
@avalonche avalonche merged commit e95a48f into flashbots:main Nov 18, 2025
10 checks passed
use reth_provider::{ProviderError, StateProvider};
use reth_revm::{State, database::StateProviderDatabase};
use reth_rpc_api::eth::{EthTxEnvError, transaction::TryIntoTxEnv};
use reth_rpc_api::eth::{EthTxEnvError, TryIntoTxEnv};

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On reth release v1.9.3, this commit has yet to be included in the release. Is there an issue with the compatibility of this change with v1.9.3?

paradigmxyz/reth@bedbfb8#diff-ec01530927c82230a16616b006dea96f9d87cea5db5d3a1697551fcf1b8e3d26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants