Skip to content
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.

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

Remove feature moonbase-runtime-benchmarks #3163

Merged
merged 2 commits into from
Jan 29, 2025

Conversation

manuelmauro
Copy link
Contributor

What does it do?

Remove unused feature flag moonbase-runtime-benchmarks.

What important points should reviewers know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

@manuelmauro manuelmauro self-assigned this Jan 29, 2025
@manuelmauro manuelmauro added B0-silent Changes should not be mentioned in any release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit labels Jan 29, 2025
Copy link
Contributor

WASM runtime size check:

Compared to target branch

Moonbase runtime: 2420 KB (no changes) 🚨

Moonbeam runtime: 2404 KB (no changes) 🚨

Moonriver runtime: 2400 KB (no changes) ✅

Compared to latest release (runtime-3401)

Moonbase runtime: 2420 KB (+388 KB compared to latest release) 🚨

Moonbeam runtime: 2404 KB (+392 KB compared to latest release) 🚨

Moonriver runtime: 2400 KB (+388 KB compared to latest release) ⚠️

Copy link
Contributor

Coverage Report

@@                                 Coverage Diff                                  @@
##           master   manuel/remove-feature-moonbase-runtime-benchmarks     +/-   ##
====================================================================================
  Coverage   74.59%                                              74.59%   0.00%     
  Files         380                                                 380             
- Lines       96035                                               96029      -6     
====================================================================================
  Hits        71632                                               71632             
- Misses      24403                                               24397      -6     
Files Changed Coverage
/node/cli/src/command.rs 17.08% (+0.14%) 🔼

Coverage generated Wed Jan 29 15:18:21 UTC 2025

@manuelmauro manuelmauro marked this pull request as ready for review January 29, 2025 15:51
@TarekkMA TarekkMA merged commit 64f1037 into master Jan 29, 2025
37 checks passed
@TarekkMA TarekkMA deleted the manuel/remove-feature-moonbase-runtime-benchmarks branch January 29, 2025 19:42
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 D3-trivial PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants