Skip to content

Add support for EIP-7702#3358

Merged
RomarQ merged 113 commits intomasterfrom
manuel/add-support-for-eip7702
Aug 27, 2025
Merged

Add support for EIP-7702#3358
RomarQ merged 113 commits intomasterfrom
manuel/add-support-for-eip7702

Conversation

@manuelmauro
Copy link
Copy Markdown
Contributor

@manuelmauro manuelmauro commented Jul 14, 2025

@manuelmauro manuelmauro self-assigned this Jul 14, 2025
@manuelmauro manuelmauro added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D9-needsaudit👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited not-breaking Does not need to be mentioned in breaking changes labels Jul 14, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jul 14, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • agent-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch manuel/add-support-for-eip7702

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

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

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 16, 2025

WASM runtime size check:

Compared to target branch

Moonbase runtime: 2424 KB (+28 KB) 🚨

Moonbeam runtime: 2572 KB (+24 KB) 🚨

Moonriver runtime: 2572 KB (+28 KB) 🚨

Compared to latest release (runtime-3800)

Moonbase runtime: 2424 KB (+380 KB compared to latest release) 🚨

Moonbeam runtime: 2572 KB (+408 KB compared to latest release) 🚨

Moonriver runtime: 2572 KB (+408 KB compared to latest release) 🚨

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 17, 2025

Coverage Report

@@                        Coverage Diff                         @@
##           master   manuel/add-support-for-eip7702      +/-   ##
==================================================================
+ Coverage   73.94%                           74.07%   +0.13%     
+ Files         405                              406       +1     
+ Lines       98357                            98987     +630     
==================================================================
+ Hits        72730                            73317     +587     
+ Misses      25627                            25670      +43     
Files Changed Coverage
/pallets/erc20-xcm-bridge/src/lib.rs 13.19% (-0.10%) 🔽
/pallets/ethereum-xcm/src/mock.rs 81.57% (+3.17%) 🔼
/pallets/moonbeam-foreign-assets/src/evm.rs 68.01% (+0.43%) 🔼
/precompiles/crowdloan-rewards/src/tests.rs 98.36% (+0.01%) 🔼
/primitives/rpc/debug/src/lib.rs 94.03% (+0.81%) 🔼
/primitives/xcm/src/ethereum_xcm.rs 95.61% (-2.76%) 🔽
/runtime/common/src/apis.rs 85.40% (+0.12%) 🔼
/runtime/common/src/impl_xcm_evm_runner.rs 61.20% (-0.16%) 🔽

Coverage generated Wed Aug 27 17:58:42 UTC 2025

Copy link
Copy Markdown
Contributor

@RomarQ RomarQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a few remarks

@librelois librelois requested a review from RomarQ August 27, 2025 18:25
Copy link
Copy Markdown
Contributor

@RomarQ RomarQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@RomarQ RomarQ merged commit 6af12f8 into master Aug 27, 2025
191 of 205 checks passed
@RomarQ RomarQ deleted the manuel/add-support-for-eip7702 branch August 27, 2025 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes breaking Needs to be mentioned in breaking changes D9-needsaudit👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants