Skip to content
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

[cumulus] parachains-common testnet constants cleaning #3134

Merged
merged 15 commits into from
Jan 31, 2024

Conversation

bkontur
Copy link
Contributor

@bkontur bkontur commented Jan 30, 2024

The parachains-common contains a lots of constants and type definitions which are used for polkadot-sdk's testnet runtimes and also for polkadot-fellows's production SP runtimes. This PR cleans parachains-common module to contain only common and generic functionality.

Testnet-specific constants have been moved to the separate module dedicated just for testnets: polkadot-sdk/cumulus/parachains/runtimes/constants/

Part of: #3054

@bkontur bkontur added R0-silent Changes should not be mentioned in any release notes T9-cumulus This PR/Issue is related to cumulus. labels Jan 30, 2024
Copy link
Contributor

@georgepisaltu georgepisaltu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM aside from one comment

cumulus/parachains/runtimes/constants/src/lib.rs Outdated Show resolved Hide resolved
@paritytech-review-bot paritytech-review-bot bot requested a review from a team January 30, 2024 23:26
@bkontur bkontur enabled auto-merge January 30, 2024 23:31
@bkontur bkontur added this pull request to the merge queue Jan 31, 2024
Merged via the queue into master with commit 4450b61 Jan 31, 2024
120 of 125 checks passed
@bkontur bkontur deleted the bko-parachains-common-consts-3054 branch January 31, 2024 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T9-cumulus This PR/Issue is related to cumulus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants