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

[FRAME] Remove V1 Module Syntax#14685

Merged
paritytech-processbot[bot] merged 6 commits intomasterfrom
oty-remove-v1-pallet-syntax
Jul 31, 2023
Merged

[FRAME] Remove V1 Module Syntax#14685
paritytech-processbot[bot] merged 6 commits intomasterfrom
oty-remove-v1-pallet-syntax

Conversation

@ggwpez
Copy link
Copy Markdown
Member

@ggwpez ggwpez commented Jul 31, 2023 •
edited
Loading

Changes:

  • Remove decl_ macro syntax
  • Move no_bound derives to own folder

Follow ups:

  • Refactor/Remove frame storage generator types

ggwpez added 5 commits July 31, 2023 10:37
@ggwpez
Remove V1 pallet syntax
b6b1003 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Remove more
5c94a9b 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
More...
fd8967b 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Move no_bound derives to own folder
b4d282a 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
fmt
1c73f0c 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez changed the title [FRAME] Remove V1 pallet syntax [FRAME] Remove V1 Module Syntax Jul 31, 2023
@ggwpez ggwpez added A3-in_progress Pull request is in progress. No review needed at this stage. 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 B1-note_worthy Changes should be noted in the release notes T1-runtime This PR/Issue is related to the topic “runtime”. labels Jul 31, 2023
@ggwpez
Keep re-exports
31dc86a 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez self-assigned this Jul 31, 2023
@ggwpez ggwpez marked this pull request as ready for review July 31, 2023 10:37
@ggwpez ggwpez requested review from a team July 31, 2023 10:37
@ggwpez ggwpez added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Jul 31, 2023
@ggwpez
Copy link
Copy Markdown
Member Author

ggwpez commented Jul 31, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 22f0669 into master Jul 31, 2023
@paritytech-processbot paritytech-processbot bot deleted the oty-remove-v1-pallet-syntax branch July 31, 2023 16:49
@ggwpez ggwpez mentioned this pull request Aug 1, 2023
Closed
@crystalin crystalin mentioned this pull request Oct 20, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

4 more reviewers

@sam0x17 sam0x17 sam0x17 approved these changes

@shawntabrizi shawntabrizi shawntabrizi approved these changes

@liamaharon liamaharon liamaharon approved these changes

@gupnik gupnik gupnik approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ggwpez ggwpez

Labels

A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the 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 T1-runtime This PR/Issue is related to the topic “runtime”.

Projects

Runtime / FRAME
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ggwpez @sam0x17 @shawntabrizi @liamaharon @gupnik