deps: bump reth and op-reth pins - #570
Merged
Merged
Conversation
sebastianst
requested review from
0x416e746f6e,
SozinM and
avalonche
as code owners
August 11, 2026 16:46
avalonche
approved these changes
Aug 11, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Rust dependency pins to keep op-rbuilder aligned with upstream reth and op-reth changes, while maintaining a single resolved copy of reth crates across the workspace.
Changes:
- Switch
reth(and relatedreth-*crates) git source fromparadigmxyz/rethtoop-rs/rethand bump the pinned revision toaef8d3ef…. - Update
op-rethcrates (and[patch.crates-io]overrides for op-alloy/op-revm) from theop-reth/v2.4.1tag to the pinnedoptimismrevisionf08ac67c…. - Regenerate
Cargo.lockso all affected packages resolve to the new git sources/revisions.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| Cargo.toml | Updates git sources and rev pins for reth/op-reth workspace dependencies and related crates.io patches. |
| Cargo.lock | Reflects the updated git sources/revisions in the resolved dependency graph. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.
Suppressed comments (1)
Makefile:81
- The pinned nightly toolchain string is duplicated across multiple
cargo +nightly-2026-04-05 ...invocations, which makes future toolchain bumps easy to do inconsistently. Consider defining a single Makefile variable (e.g.NIGHTLY_TOOLCHAIN) and referencing it in thelint/fmtrecipes.
cargo +nightly-2026-04-05 fmt --all -- --check
cargo +nightly-2026-04-05 clippy --all-targets --all-features -- -D warnings
sebastianst
force-pushed
the
seb/bump-op-reth
branch
from
August 12, 2026 14:34
c92c376 to
7f1a79d
Compare
avalonche
enabled auto-merge (squash)
August 12, 2026 15:34
avalonche
disabled auto-merge
August 12, 2026 15:34
Collaborator
|
hey! could you add verified signatures to your commits so we can merge these changes? |
- reth: rev f2eecc65 (paradigmxyz) -> rev aef8d3ef (op-rs), matching the reth pin of the op-reth revision below. op-rs/reth is the fork op-reth pins its reth from; this rev is upstream f2eecc65 plus one additive commit (trie: expose payload state root receiver). Using the same declaration keeps cargo resolving a single copy of every reth crate. - op-reth: op-reth/v2.4.1 -> rev f08ac67c (workspace deps + crates-io patches) No source changes required. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
sebastianst
force-pushed
the
seb/bump-op-reth
branch
from
August 12, 2026 15:55
7f1a79d to
955d4ba
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 Summary
redundant_field_namesanddouble_must_uselints workspace-wide.No source changes required.
💡 Motivation and Context
Maintenance upgrade to keep up with latest upstream changes.
The dependency bump raises the workspace MSRV to Rust 1.96. The moving nightly toolchain currently emits
redundant_field_namesanddouble_must_usediagnostics for the updated dependency integration, including RPC methods generated byjsonrpsee. Both are style-only lints. Allowing them globally keeps CI on the moving nightly channel without requiring a manually maintained toolchain pin, while retaining-D warningsfor all other diagnostics.✅ I have completed the following steps:
make lintmake test🤖 Co-created with openai-codex/gpt-5.6-sol