chore: prep apps v1.2.1#3393
Conversation
Drop the local replace directives from apps/evm and apps/testapp and pin the published upstreams: - apps/evm: ev-node v1.2.1, core v1.1.0, execution/evm v1.1.0 - apps/testapp: ev-node v1.2.1, core v1.1.0 Release-ready go.mod for tagging apps/evm/v1.2.1 and apps/testapp/v1.2.1. Local cross-module dev is handled by go.work. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe EVM and test application Go modules remove local ev-node replacements and update their ev-node dependency versions to released versions. ChangesApplication dependency updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Checkov (3.3.8)apps/evm/go.modTraceback (most recent call last): apps/testapp/go.modTraceback (most recent call last): Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The latest Buf updates on your PR. Results from workflow CI / buf-check (pull_request).
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3393 +/- ##
=======================================
Coverage 62.34% 62.34%
=======================================
Files 120 120
Lines 13454 13454
=======================================
Hits 8388 8388
- Misses 4125 4128 +3
+ Partials 941 938 -3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Release-ready go.mods for tagging apps/evm/v1.2.1 and apps/testapp/v1.2.1.
Changes
Drop the local
replaceblocks and pin published upstreams:ev-node v1.2.1,core v1.1.0,execution/evm v1.0.1 → v1.1.0ev-node v1.1.3 → v1.2.1,core v1.1.0Local cross-module dev relies on
go.work(go.work.examplecovers all modules).Verification
go build ./...pass resolving published modules (no replace)go mod tidyidempotent on bothapps/loadgen,execution/evm/test,test/e2e) still buildFinal code step of the v1.2.1 chain: core/v1.1.0 → v1.2.1 → execution/evm/v1.1.0 → apps/*/v1.2.1. (
apps/grpcremoved in #3380 — not released this cycle.)🤖 Generated with Claude Code
Summary by CodeRabbit