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

[FRAME] Remove V1 Module Syntax #14685

Merged
merged 6 commits into from
Jul 31, 2023
Merged

[FRAME] Remove V1 Module Syntax #14685

merged 6 commits into from
Jul 31, 2023

Conversation

ggwpez
Copy link
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
Remove V1 pallet syntax
b6b1003 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez
Remove more
5c94a9b 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez
More...
fd8967b 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez
Move no_bound derives to own folder
b4d282a 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez
fmt
1c73f0c 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@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 <[email protected]>
@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
Member Author

ggwpez commented Jul 31, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 22f0669 into master Jul 31, 2023
15 of 16 checks passed
@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
@JesseAbram JesseAbram mentioned this pull request Oct 26, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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

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