Skip to content

Add support for EIP-7623 #3441

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
RomarQ merged 12 commits into from
Sep 10, 2025
Merged

Add support for EIP-7623 #3441

RomarQ merged 12 commits into from
Sep 10, 2025

Conversation

@manuelmauro
Copy link
Contributor

@manuelmauro manuelmauro commented Sep 8, 2025

What does it do?

Add support for EIP-7623.

changes on evm: moonbeam-foundation/evm@b3d5b09

No changes required on: frontier, moonkit, moonbeam. Just updated the dependencies.

@coderabbitai
Copy link

coderabbitai bot commented Sep 8, 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-eip-7623

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.

@manuelmauro manuelmauro marked this pull request as ready for review September 8, 2025 14:54
@manuelmauro manuelmauro requested a review from RomarQ September 8, 2025 14:57
@RomarQ RomarQ added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes dependencies Pull requests that update a dependency file not-breaking Does not need to be mentioned in breaking changes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit labels Sep 8, 2025
RomarQ
RomarQ previously approved these changes Sep 8, 2025
View reviewed changes
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2025 •
edited
Loading

WASM runtime size check:

Compared to target branch

Moonbase runtime: 2600 KB (no changes) ✅

Moonbeam runtime: 2776 KB (+8 KB) ⚠️

Moonriver runtime: 2768 KB (no changes) ✅

Compared to latest release (runtime-3900)

Moonbase runtime: 2600 KB (+240 KB compared to latest release) ⚠️

Moonbeam runtime: 2776 KB (+296 KB compared to latest release) ⚠️

Moonriver runtime: 2768 KB (+288 KB compared to latest release) ⚠️

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2025 •
edited
Loading

Coverage Report

@@                        Coverage Diff                         @@
##           master   manuel/add-support-for-eip-7623     +/-   ##
==================================================================
  Coverage   74.27%                            74.27%   0.00%     
  Files         396                               396             
  Lines       95445                             95445             
==================================================================
  Hits        70883                             70883             
  Misses      24562                             24562
Files Changed Coverage

Coverage generated Wed Sep 10 11:59:26 UTC 2025

RomarQ
RomarQ reviewed Sep 10, 2025
View reviewed changes
RomarQ
RomarQ reviewed Sep 10, 2025
View reviewed changes
RomarQ
RomarQ reviewed Sep 10, 2025
View reviewed changes
RomarQ
RomarQ previously approved these changes Sep 10, 2025
View reviewed changes
@RomarQ RomarQ merged commit 5f6529f into master Sep 10, 2025
167 of 172 checks passed
@RomarQ RomarQ deleted the manuel/add-support-for-eip-7623 branch September 10, 2025 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RomarQ RomarQ RomarQ approved these changes

Assignees

No one assigned

Labels

B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit dependencies Pull requests that update a dependency file not-breaking Does not need to be mentioned in breaking changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@manuelmauro @RomarQ