Skip to content

chore: use released ev-node module in apps - #3427

Merged
tac0turtle merged 1 commit into
mainfrom
agent/remove-app-module-replaces
Aug 11, 2026
Merged

chore: use released ev-node module in apps#3427
tac0turtle merged 1 commit into
mainfrom
agent/remove-app-module-replaces

Conversation

@tac0turtle

@tac0turtletac0turtle commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Update apps/evm and apps/testapp to use the published github.com/evstack/ev-node v1.2.3 module.
  • Remove their local root-module replace directives.
  • Record the v1.2.3 module checksums.

Why

The root v1.2.3 release is now published, so these app modules can resolve the released dependency instead of relying on the monorepo checkout. Test-only local replacements remain unchanged.

Validation

  • go test ./... in apps/evm
  • go test ./... in apps/testapp
  • go mod verify in both modules

Summary by CodeRabbit

  • Maintenance
    • Updated the application integration to use ev-node v1.2.3.
    • Removed local development overrides so builds use the published dependency version.

@coderabbitai

coderabbitaiBot commented Aug 11, 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 Plus

Run ID: 0eda40d9-9975-413f-a3cc-d84a98ab18f4

📥 Commits

Reviewing files that changed from the base of the PR and between d76e0df and d4734d7.

⛔ 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 modules upgrade github.com/evstack/ev-node from v1.2.2 to v1.2.3 and remove local replacement directives.

Changes

ev-node dependency update

Layer / File(s)Summary
Update application module requirements
apps/evm/go.mod, apps/testapp/go.mod
Both modules require github.com/evstack/ev-node v1.2.3 and no longer use the local replacement directive.

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

Possibly related PRs

Suggested reviewers:julienrbrt, chatton

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly states that the applications now use the released ev-node module and follows the conventional commit format.
Description check✅ PassedThe description explains what changed, why it changed, and how the changes were validated.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck 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 agent/remove-app-module-replaces

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⏩ skippedAug 11, 2026, 6:47 AM

@tac0turtle
tac0turtle marked this pull request as ready for review August 11, 2026 06:47
@codecov

codecovBot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.43%. Comparing base (d47195a) to head (d4734d7).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #3427 +/- ##
==========================================
- Coverage 62.45% 62.43% -0.02% 
==========================================
Files 121 121 Lines 13469 13469 ==========================================
- Hits 8412 8410 -2 - Misses 4116 4118 +2 
Partials 941 941 
FlagCoverage Δ
combined62.43% <ø> (-0.02%)⬇️

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 5c88745 into mainAug 11, 2026
32 checks passed
@tac0turtle
tac0turtle deleted the agent/remove-app-module-replaces branch August 11, 2026 06:54
Sign up for freeto 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

@tac0turtle