Skip to content

chore: prep apps v1.2.1#3393

Merged
tac0turtle merged 1 commit into
mainfrom
chore/release-apps-1.2.1
Jul 20, 2026
Merged

chore: prep apps v1.2.1#3393
tac0turtle merged 1 commit into
mainfrom
chore/release-apps-1.2.1

Conversation

@tac0turtle

@tac0turtle tac0turtle commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Release-ready go.mods for tagging apps/evm/v1.2.1 and apps/testapp/v1.2.1.

Changes

Drop the local replace blocks and pin published upstreams:

  • apps/evm: ev-node v1.2.1, core v1.1.0, execution/evm v1.0.1 → v1.1.0
  • apps/testapp: ev-node v1.1.3 → v1.2.1, core v1.1.0

Local cross-module dev relies on go.work (go.work.example covers all modules).

Verification

  • Both apps go build ./... pass resolving published modules (no replace)
  • go mod tidy idempotent on both
  • Untouched modules (apps/loadgen, execution/evm/test, test/e2e) still build

Final 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/grpc removed in #3380 — not released this cycle.)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated core platform components to newer released versions.
    • Removed local development overrides so applications use the published component packages.
    • Updated the test application to align with the latest platform release.

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>
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f8c9c414-15b5-40a6-89f9-990e35554b26

📥 Commits

Reviewing files that changed from the base of the PR and between b516561 and e799f51.

⛔ Files ignored due to path filters (2)
  • apps/evm/go.sum is excluded by !**/*.sum
  • apps/testapp/go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • apps/evm/go.mod
  • apps/testapp/go.mod

📝 Walkthrough

Walkthrough

The EVM and test application Go modules remove local ev-node replacements and update their ev-node dependency versions to released versions.

Changes

Application dependency updates

Layer / File(s) Summary
Released ev-node dependency resolution
apps/evm/go.mod, apps/testapp/go.mod
The EVM module updates execution/evm to v1.1.0, the test app updates ev-node to v1.2.1, and both remove local replacement directives.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: julienrbrt, chatton

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately reflects the release-prep version bump and replace-block removals.
Description check ✅ Passed The description covers the overview, changes, and verification well enough for the template requirements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/release-apps-1.2.1

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.mod

Traceback (most recent call last):
File "/usr/local/bin/checkov", line 2, in
from checkov.main import Checkov
ModuleNotFoundError: No module named 'checkov'

apps/testapp/go.mod

Traceback (most recent call last):
File "/usr/local/bin/checkov", line 2, in
from checkov.main import Checkov
ModuleNotFoundError: No module named 'checkov'


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

The latest Buf updates on your PR. Results from workflow CI / buf-check (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed⏩ skipped✅ passed⏩ skippedJul 20, 2026, 11:33 AM

@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.34%. Comparing base (b516561) to head (e799f51).

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     
Flag Coverage Δ
combined 62.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tac0turtle
tac0turtle merged commit 8dc1619 into main Jul 20, 2026
33 checks passed
@tac0turtle
tac0turtle deleted the chore/release-apps-1.2.1 branch July 20, 2026 12:08
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.

1 participant