Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
feat: introduce pallet-parameters to Westend to parameterize inflation #4938
feat: introduce pallet-parameters to Westend to parameterize inflation #4938
Changes from all commits
81da185
e84a472
e8d2ec5
77a28cc
b010d0b
30caa8c
08b9287
282aa21
ff9e782
90f031d
ae01d45
aa563f5
ae0a72d
451f622
849604c
e866cd9
833d209
2ea077f
8843d73
704e13c
4884a48
c7ee56c
ede367a
fdeaacd
c866a69
7ce4449
e8037de
8d62d46
810b2b0
d8760c7
026dcb6
2e128ac
8a6d11e
6ab2d43
e993b59
e520f23
28c1c2d
98d7d91
4f6316a
b972461
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit but in this runtime it is implicit that the configs/impls are for Westend relay chain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this fn is not used anywhere but in the tests, does it need
#[allow(dead_code)]
to avoid compilation warnings?I'd probably have moved this implementation and the tests into a
mod test_inflation {}
as we probably dont use this code in the runtime anymore.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit but moving this code under a mod gated by cfg(test) also make sense given that in the prdoc, it says that the old era payout impl has been removed :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it currently lives under
in L265