Skip to content

Conversation

@RomarQ
Copy link
Contributor

@RomarQ RomarQ commented Sep 4, 2025

What does it do?

Reverts most of the changes included in Bridge setup (Stagenet <-> Betanet)

What important points should reviewers know?

Since we now have a proper testnet for testing the bridge between Moonbeam and Moonriver, there is no longer the need to have the bridging functionality in the moonbase-runtime.

@RomarQ RomarQ self-assigned this Sep 4, 2025
@RomarQ RomarQ requested review from a team as code owners September 4, 2025 12:03
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025

WASM runtime size check:

Compared to target branch

Moonbase runtime: 2676 KB (no changes) ✅

Moonbeam runtime: 2848 KB (no changes) ✅

Moonriver runtime: 2844 KB (no changes) ✅

Compared to latest release (runtime-3900)

Moonbase runtime: 2676 KB (+316 KB compared to latest release) ⚠️

Moonbeam runtime: 2848 KB (+368 KB compared to latest release) ⚠️

Moonriver runtime: 2844 KB (+364 KB compared to latest release) ⚠️

@RomarQ RomarQ marked this pull request as draft September 4, 2025 12:13
@RomarQ RomarQ added 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 labels Sep 4, 2025
@RomarQ RomarQ added the not-breaking Does not need to be mentioned in breaking changes label Sep 4, 2025
@RomarQ RomarQ marked this pull request as ready for review September 4, 2025 14:11
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025

Coverage Report

@@                                 Coverage Diff                                  @@
##           master   rq/remove-bridge-features-from-moonbase-runtimes      +/-   ##
====================================================================================
+ Coverage   74.09%                                             74.10%   +0.01%     
- Files         406                                                404       -2     
- Lines       98579                                              98522      -57     
====================================================================================
- Hits        73037                                              73005      -32     
- Misses      25542                                              25517      -25     
Files Changed Coverage
/runtime/moonbase/src/lib.rs 54.41% (-0.81%) 🔽
/runtime/moonbase/tests/common/mod.rs 97.31% (+1.91%) 🔼

Coverage generated Thu Sep 4 14:52:11 UTC 2025

Copy link
Contributor

@manuelmauro manuelmauro left a comment

Choose a reason for hiding this comment

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

LGTM. Do we need any migration for this?

@RomarQ
Copy link
Contributor Author

RomarQ commented Sep 5, 2025

LGTM. Do we need any migration for this?

I would say that we don't need a migration, those pallets were never included in Alphanet, only on betanet and stagenet.

@RomarQ RomarQ merged commit 6d9e07a into master Sep 8, 2025
41 checks passed
@RomarQ RomarQ deleted the rq/remove-bridge-features-from-moonbase-runtimes branch September 8, 2025 12:31
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 D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants