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

Adds ConstGet trait and impl in parameter types #13391

Closed
wants to merge 3 commits into from
Closed

Adds ConstGet trait and impl in parameter types #13391

wants to merge 3 commits into from

Conversation

gupnik
Copy link
Contributor

@gupnik gupnik commented Feb 15, 2023 •
edited
Loading

Fixes 1st and 6th tasks for paritytech/polkadot-sdk#303

  1. Adds ConstGet<T> trait
  2. Update parameter_types! macro to generate ConstGet<T> on const

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Feb 15, 2023

User @gupnik, please sign the CLA here.

@dmitry-markin dmitry-markin added 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 labels Mar 7, 2023
@stale
Copy link

stale bot commented Apr 6, 2023

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A3-stale label Apr 6, 2023
@gupnik gupnik closed this Apr 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. A3-stale 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
Runtime / FRAME
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gupnik @dmitry-markin @codekitz