-
Notifications
You must be signed in to change notification settings - Fork 7
chore: bump to reth 1.9.4 #90
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
Conversation
Summary of ChangesHello @tac0turtle, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request performs a routine maintenance update by bumping the versions of all Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
Claude finished @tac0turtle's task —— View job Code Review for Reth 1.9.3 Dependency BumpI've completed my review of this pull request that updates Reth dependencies from v1.9.2 to v1.9.3. Review Tasks:
Critical Issues:1. PR Title Mismatch
Code Quality Assessment:✅ Strengths:
✅ Dependency Management:
Security Analysis:✅ Positive Changes:
🔍 New Dependencies: Added several new crates that should be monitored:
Performance Considerations:✅ Expected Improvements:
Test Coverage:# Verify build succeeds
make build
# Run comprehensive test suite
make test
# Test Evolve-specific functionality
make test-integration
# Verify linting passes
make lintCompatibility Assessment:✅ Low Risk Update: This appears to be a minor version bump with:
Recommendations:
Overall Assessment: ✅ Approve with Minor FixThis is a well-executed dependency update that follows established patterns and best practices. The only issue is the incorrect version number in the title. The changes appear safe and maintain compatibility with the existing Evolve-specific customizations. |
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.
Code Review
This pull request bumps the reth dependency version. However, there's a mismatch between the version in the pull request title (1.9.4) and the version in the code changes (v1.9.3). I've suggested updating the version in Cargo.toml to v1.9.4 to align with the PR's intent.
| reth-chainspec = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-cli = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-cli-util = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-rpc-eth-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-tracing-otlp = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-node-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-node-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-errors = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-optimism-node = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-optimism-cli = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-optimism-rpc = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-optimism-forks = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-optimism-chainspec = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-optimism-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-optimism-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-trie-db = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-trie-common = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-rpc-eth-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-primitives-traits = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-provider = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-storage-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-tracing = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-transaction-pool = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-network = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-network-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-chain-state = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-ethereum = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-ethereum-cli = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-basic-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-engine-local = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-engine-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-ethereum-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-ethereum-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-e2e-test-utils = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-evm = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-evm-ethereum = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-execution-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-node-core = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-node-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-payload-builder-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-payload-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-ethereum-forks = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-revm = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-rpc-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-rpc-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
| reth-rpc-engine-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3" } | ||
|
|
||
| ev-revm = { path = "crates/ev-revm" } | ||
|
|
||
|
|
||
| # Consensus dependencies | ||
| reth-consensus = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.2", default-features = false } | ||
| reth-consensus-common = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.2", default-features = false } | ||
| reth-ethereum-consensus = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.2", default-features = false } | ||
| reth-consensus = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3", default-features = false } | ||
| reth-consensus-common = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3", default-features = false } | ||
| reth-ethereum-consensus = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3", default-features = false } | ||
|
|
||
| # Test dependencies | ||
| reth-testing-utils = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.2", default-features = false } | ||
| reth-db = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.2", default-features = false } | ||
| reth-tasks = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.2", default-features = false } | ||
| reth-testing-utils = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3", default-features = false } | ||
| reth-db = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3", default-features = false } | ||
| reth-tasks = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.3", default-features = false } |
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.
The pull request title indicates a bump to reth version 1.9.4, but the dependencies are being updated to v1.9.3. To align with the PR's intent, all reth dependencies should be updated to v1.9.4.
reth-chainspec = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-cli = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-cli-util = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-rpc-eth-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-tracing-otlp = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-node-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-node-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-errors = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-optimism-node = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-optimism-cli = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-optimism-rpc = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-optimism-forks = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-optimism-chainspec = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-optimism-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-optimism-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-trie-db = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-trie-common = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-rpc-eth-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-primitives-traits = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-provider = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-storage-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-tracing = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-transaction-pool = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-network = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-network-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-chain-state = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-ethereum = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-ethereum-cli = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-basic-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-engine-local = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-engine-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-ethereum-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-ethereum-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-e2e-test-utils = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-evm = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-evm-ethereum = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-execution-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-node-core = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-node-types = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-payload-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-payload-builder-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-payload-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-primitives = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-ethereum-forks = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-revm = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-rpc-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-rpc-builder = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
reth-rpc-engine-api = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4" }
ev-revm = { path = "crates/ev-revm" }
# Consensus dependencies
reth-consensus = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4", default-features = false }
reth-consensus-common = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4", default-features = false }
reth-ethereum-consensus = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4", default-features = false }
# Test dependencies
reth-testing-utils = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4", default-features = false }
reth-db = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4", default-features = false }
reth-tasks = { git = "https://github.com/paradigmxyz/reth.git", tag = "v1.9.4", default-features = false }
|
we will most likely downgrade, reth 1.9.4 has lots of issues for us and there are open issues on reth about other issues with 1.9 |
Description
Type of Change
Related Issues
Fixes #(issue)
Checklist
Testing
Additional Notes