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

Switch construct_runtime! to use default syntax #116

Closed
bkchr opened this issue Dec 8, 2023 · 2 comments
Closed

Switch construct_runtime! to use default syntax #116

bkchr opened this issue Dec 8, 2023 · 2 comments
Labels
good first issue Good for newcomers

Comments

@bkchr
Copy link
Contributor

bkchr commented Dec 8, 2023

Currently we are still declaring for every pallet all the parts we want to use for it:

Babe: pallet_babe::{Pallet, Call, Storage, Config<T>, ValidateUnsigned} = 2,

Instead we can just write:

Babe: pallet_babe = 2,

There is no real reason to continue using the old syntax of always declaring all parts and it is also more error prone.

@bkchr bkchr added the good first issue Good for newcomers label Dec 8, 2023
@Aideepakchaudhary
Copy link

Aideepakchaudhary commented Mar 18, 2024

@bkchr I think this issue is solved here: #152

@bkchr
Copy link
Contributor Author

bkchr commented Mar 18, 2024

Yes! thank you

@bkchr bkchr closed this as completed Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants