Skip to content

Update PolkaVM to latest version#11650

Merged
s0me0ne-unkn0wn merged 5 commits intomasterfrom
s0me0ne/bump-polkavm
Apr 7, 2026
Merged

Update PolkaVM to latest version#11650
s0me0ne-unkn0wn merged 5 commits intomasterfrom
s0me0ne/bump-polkavm

Conversation

@s0me0ne-unkn0wn
Copy link
Copy Markdown
Contributor

No description provided.

@s0me0ne-unkn0wn s0me0ne-unkn0wn requested review from a team and koute as code owners April 6, 2026 10:49
@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 6, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedcargo/​polkavm@​0.31.0 ⏵ 0.33.19710093100100
Updatedcargo/​polkavm-common@​0.31.0 ⏵ 0.33.010010093100100
Updatedcargo/​polkavm-derive@​0.31.0 ⏵ 0.33.010010093100100
Updatedcargo/​polkavm-linker@​0.31.0 ⏵ 0.33.099100100100100

View full report

@s0me0ne-unkn0wn s0me0ne-unkn0wn added the T0-node This PR/Issue is related to the topic “node”. label Apr 6, 2026
@s0me0ne-unkn0wn
Copy link
Copy Markdown
Contributor Author

/cmd prdoc --audience node_dev,runtime_dev --bump patch

Comment thread substrate/client/executor/polkavm/src/lib.rs Outdated
@s0me0ne-unkn0wn s0me0ne-unkn0wn added this pull request to the merge queue Apr 7, 2026
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

revive does not use the pvm version from the main Cargo.toml. So those changes here should not be necessary.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

revive does not use the pvm version from the main Cargo.toml

Seems like that has changed?

polkavm = { workspace = true }
polkavm-common = { workspace = true, features = ["alloc"] }

Copy link
Copy Markdown
Contributor Author

@s0me0ne-unkn0wn s0me0ne-unkn0wn Apr 7, 2026

Choose a reason for hiding this comment

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

Oh I even approved it 🙃 #11096

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ahh I didn't know that. We kept it separate when pvm was still breaking semantics with new versions. I guess its fine to sync it now.

Merged via the queue into master with commit 5e1f597 Apr 7, 2026
327 of 338 checks passed
@s0me0ne-unkn0wn s0me0ne-unkn0wn deleted the s0me0ne/bump-polkavm branch April 7, 2026 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T0-node This PR/Issue is related to the topic “node”.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants