-
Notifications
You must be signed in to change notification settings - Fork 710
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
[tracking] Migrate pallets to umbrella crate #6504
Comments
Can I pick this up @re-gius Thinking this should be a tracking issue as it involves more than one crate but happy to do it! |
Yes, it's going to be a tracking issue. You are more than welcome to start working at some pallets/crates, ideally one or two per PR. |
Are there still some pallets to work on or are you doing all of them? |
I am not fast enough, will try another issue. |
Some updates:
|
PRs for this Issue:
|
bennethxyz runcomet can you please read the new checklist that I put above, and make sure it is clear? Feel to ask more questions from me and @re-gius. Once clear, please update the PRs (most of them are not 100% correct). |
Migrate all pallets to
polkadot-sdk-frame
by adding types and preludes to it when necessary. As a reference, you can look into #5995.The guidelines of the umbrella crate are as follows:
polkadot-sdk/substrate/frame/src/lib.rs
Lines 134 to 150 in 06a68be
✅ Checklist:
polkadot-sdk-frame
to theCargo.toml
frame_support
,sp_runtime
and similar, and replace with the appropriateprelude::*
deps
Cargo.toml
to remove all unneeded dependencies.👉 If you want to work on this: Please check that there is not already a merge request for the pallet that you want to work on (here or in the comments below). Please pick only one or a small set of pallets - not all at once. This keeps review times low. Add a comment below to announce the pallets you're working on.
Completed:
pallet-proxy
andpallet-multisig
#5995pallet-proxy
andpallet-multisig
#5995In progress:
pallet-babe
&pallet-staking-reward-curve
#6412pallet-babe
&pallet-staking-reward-curve
#6412To do:
The text was updated successfully, but these errors were encountered: