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

pallet-referenda: make the pallet instantiable #11089

Merged
merged 1 commit into from
Mar 24, 2022
Merged

pallet-referenda: make the pallet instantiable #11089

merged 1 commit into from
Mar 24, 2022

Conversation

koushiro
Copy link
Contributor

@koushiro koushiro commented Mar 22, 2022

Signed-off-by: koushiro [email protected]

related issue: #11056

polkadot address: 15XuanNimo5951s3RjFTPX1AvYVnCwfr3SDhb3AV4fQF3LpK

@koushiro koushiro changed the title pallet-referenda: make the pallet instanceable pallet-referenda: make the pallet instantiable Mar 22, 2022
@shawntabrizi shawntabrizi added 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 labels Mar 22, 2022
Comment on lines +87 to +88
use self::branch::{BeginDecidingBranch, OneFewerDecidingBranch, ServiceBranch};
pub use self::{
Copy link
Member

Choose a reason for hiding this comment

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

why self?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Adding self or crate makes it easier to distinguish module name from external crate name, I think

@shawntabrizi
Copy link
Member

delivered a tip in the conviction-voting pr

@shawntabrizi
Copy link
Member

bot merge

@shawntabrizi shawntabrizi merged commit 666f39b into paritytech:master Mar 24, 2022
@koushiro koushiro deleted the referenda-instance branch March 24, 2022 16:38
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
DaviRain-Su pushed a commit to octopus-network/substrate that referenced this pull request Aug 23, 2022
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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.

3 participants