Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

pallets: implement Default for GenesisConfig in no_std #2624

Merged

Conversation

michalkucharczyk
Copy link
Contributor

Implementation of Default trait for pallets enabled for no_std environments.

This change is follow-up of: paritytech/substrate#14108
It is a step towards: paritytech/polkadot-sdk#25

Companion for: paritytech/substrate#14107

@michalkucharczyk michalkucharczyk added B0-silent Changes should not be mentioned in any release notes A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels May 23, 2023
@michalkucharczyk
Copy link
Contributor Author

bot fmt

@paritytech paritytech deleted a comment from command-bot bot May 23, 2023
@michalkucharczyk
Copy link
Contributor Author

bot clean

@michalkucharczyk michalkucharczyk marked this pull request as ready for review May 23, 2023 16:17
@paritytech-ci paritytech-ci requested a review from a team May 23, 2023 16:18
@michalkucharczyk
Copy link
Contributor Author

bot rebase

@paritytech-processbot
Copy link

Rebased

@michalkucharczyk
Copy link
Contributor Author

bot rebase

@paritytech-processbot
Copy link

Rebased

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 41cc2a7 into master May 25, 2023
@paritytech-processbot paritytech-processbot bot deleted the mku-pallets-default-impl-for-genesis-config branch May 25, 2023 23:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants