Skip to content

Conversation

@manuelmauro
Copy link
Contributor

@manuelmauro manuelmauro commented Mar 20, 2025

What does it do?

It updates polkadot-sdk to version stable2412.

⚠️ Breaking Changes ⚠️

  • Removes precompile StorageCleanerPrecompile.

Review Checklist

Important PRs

Tanssi's Experience Upgrading to Polkadot SDK 2412

Main PRs

Not Important But Good To Know

  • extension weights - terminology change: unsigned => bare. Also refactors traits such as CreateSignedTransaction
  • deprecate RuntimeString - Remove sp_runtime::RuntimeString and replace with Cow<'static, str> or String depending on se case
  • system_version - rename RuntimeVersion::state_version to system_version. Setting system_version = 2 will make xtrinsics use StateVersion::V1 instead of V0, but we don't do that, and neither does parity
  • remove ttl field - not removed, instead deprecated and ignored

New Types In Pallet Config

@manuelmauro
Copy link
Contributor Author

We need to rename MM_MigrateToLatestXcmVersion to MM_MigrateToV5XcmVersion. Otherwise, the migration will not run.

Enabled in 5d44ae8

@RomarQ
Copy link
Contributor

RomarQ commented May 13, 2025

We need to merge the changes from master

@manuelmauro
Copy link
Contributor Author

We need to merge the changes from master

Merged in c83782c

Copy link
Contributor

@stiiifff stiiifff left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@manuelmauro manuelmauro merged commit a84f80a into master May 13, 2025
37 checks passed
@manuelmauro manuelmauro deleted the update-polkadot-stable2412-rc2 branch May 13, 2025 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A8-mergeoncegreen Pull request is reviewed well. 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.

5 participants