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

Add a deprecation warning to the old runtime GenesisConfig #14224

Merged

Conversation

juangirini
Copy link
Contributor

Relates to paritytech/polkadot-sdk#175

This PR adds deprecation warning when using the runtime level GenesisConfig which has been renamed to RuntimeGenesisConfig.

cumulus companion: paritytech/cumulus#2636
polkadot companion: paritytech/polkadot#7288

@juangirini juangirini added A3-in_progress Pull request is in progress. No review needed at this stage. C3-medium PR touches the given topic and has a medium 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 May 25, 2023
Base automatically changed from jg/14065-deprecation-genesisconfig-runtimegenesisconfig to master May 26, 2023 09:42
@juangirini juangirini changed the title [WIP] add a deprecation warning to the old runtime GenesisConfig Add a deprecation warning to the old runtime GenesisConfig May 26, 2023
@juangirini juangirini marked this pull request as ready for review May 26, 2023 13:37
@juangirini juangirini requested review from a team May 26, 2023 13:37
@juangirini juangirini 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 May 26, 2023
@juangirini
Copy link
Contributor Author

juangirini commented May 26, 2023

This PR should be merged after polkadot-developers/substrate-docs#1996 has been addressed
It no longer applies. We are trying a different Deprecation process

@ggwpez ggwpez removed the A0-please_review Pull request needs code review. label Jun 14, 2023
@juangirini
Copy link
Contributor Author

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 6e1a5c4 into master Jun 16, 2023
@paritytech-processbot paritytech-processbot bot deleted the jg/14065-deprecation-warning-genesisconfig branch June 16, 2023 10:54
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
…h#14224)

* soft deprecate genesisconfig

* temporarily add a deprecation attr

* update tests

* update mocks

* update genesis config

* update genesis config

* update genesis config

* update genesis config

* remove deprecation warning

* add a deprecation warning to the old runtime GenesisConfig

* add a date to the deprecation message

---------

Co-authored-by: parity-processbot <>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B1-note_worthy Changes should be noted in the release notes C3-medium PR touches the given topic and has a medium 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
None yet
Development

Successfully merging this pull request may close these issues.

10 participants