Skip to content

Depend on MBF's ethereum fork#3682

Merged
manuelmauro merged 1 commit intomasterfrom
manuel/depend-on-ethereum-mbf-fork
Feb 25, 2026
Merged

Depend on MBF's ethereum fork#3682
manuelmauro merged 1 commit intomasterfrom
manuel/depend-on-ethereum-mbf-fork

Conversation

@manuelmauro
Copy link
Contributor

What does it do?

Switch the ethereum crate dependency from crates.io (0.18.2) to the Moonbeam Foundation fork (moonbeam-polkadot-stable2506 branch).

Details

  • Replace the crates.io ethereum dependency with git+https://github.com/moonbeam-foundation/ethereum?branch=moonbeam-polkadot-stable2506 in the workspace Cargo.toml.
  • Cargo.lock updates reflect the new ethereum source and bumped commit hashes across frontier, evm, and moonkit forks (all on moonbeam-polkadot-stable2506).
  • No code changes; only dependency source and lockfile updates.

@manuelmauro manuelmauro self-assigned this Feb 25, 2026
@manuelmauro manuelmauro added B0-silent Changes should not be mentioned in any release notes D2-notlive PR doesn't change runtime code (so can't be audited) not-breaking Does not need to be mentioned in breaking changes labels Feb 25, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 05ce4c8 and 8e1ef51.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml

📝 Walkthrough

Walkthrough

The changes modify Cargo.toml dependency configurations: the Frontier ethereum crate is switched from a version constraint to a git source with an explicit branch, and the frame-metadata feature list is reformatted to a multi-line array.

Changes

Cohort / File(s) Summary
Cargo Manifest
Cargo.toml
Updated Frontier ethereum dependency from version spec to git source with explicit branch; reformatted frame-metadata feature array to multi-line syntax; added end-of-file newline.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: switching the ethereum crate dependency to Moonbeam Foundation's fork, which aligns with the primary modification in the changeset.
Description check ✅ Passed The description provides relevant context about the dependency change, explaining what is being switched, why, and what files are affected, which directly relates to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch manuel/depend-on-ethereum-mbf-fork

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 and usage tips.

@github-actions
Copy link
Contributor

Coverage Report

@@                          Coverage Diff                          @@
##           master   manuel/depend-on-ethereum-mbf-fork     +/-   ##
=====================================================================
  Coverage   77.10%                               77.10%   0.00%     
  Files         389                                  389             
  Lines       76943                                76943             
=====================================================================
  Hits        59323                                59323             
  Misses      17620                                17620             
Files Changed Coverage

Coverage generated Wed Feb 25 10:55:56 UTC 2026

@manuelmauro manuelmauro merged commit ee9498c into master Feb 25, 2026
79 of 83 checks passed
@manuelmauro manuelmauro deleted the manuel/depend-on-ethereum-mbf-fork branch February 25, 2026 13:33
@arturgontijo arturgontijo mentioned this pull request Mar 5, 2026
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B0-silent Changes should not be mentioned in any release notes D2-notlive PR doesn't change runtime code (so can't be audited) not-breaking Does not need to be mentioned in breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants