-
Notifications
You must be signed in to change notification settings - Fork 257
Stable2503 #3735
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
Stable2503 #3735
Conversation
🛡️ Immunefi PR ReviewsWe noticed that your project isn't set up for automatic code reviews. If you'd like this PR reviewed by the Immunefi team, you can request it manually using the link below: Once submitted, we'll take care of assigning a reviewer and follow up here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Can we add the detailed mainnet deployment strategy to the PR description?
|
@jfrank-summit need another review. Recent farmerless dev changes are now included and fixed the RPC provider that was failing in the merge-queue |
Migrate from stable2412 to stable 2503 latest. Our fork is updated accordingly with necessary patches applied on top.
Some of the patches can be pushed upstream and will send a patch once I'm finished with migration.
PR aims to avoid behavior change as much as possible. New features are not included if not straightforward and maybe added in follow-up PRs
Changes
DecodeWithMemTrackingmarker trait that needs to be added to our types that cross the runtime boundary to client.ProofSizewhich it uses to get the measure proof size of the ext rather than an estimation unlike before. More here polkadot-evm/frontier@d5755b5if_elseutility call is added where ext contain default Call and Fallback back if default fails. This would be tricky to handle for object mapping and left a TODO for later PRswasm32v1-none. I did not want to take migration to new target yet until we are the sure the rest of the upgrade is compatible.TODOs
I'm leaving this PR as draft until Devnet TODO is complete but please do review in the meantimeMigration plan is listed here - https://www.notion.so/subspacelabs/Stable-2503-2cc0bae47e9580cd9642ca225d91797c
Code contributor checklist: