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

Only one ParaId variable in the Parachain Template #6744

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shawntabrizi
Copy link
Member

Many problems can occur when building and testing a Parachain caused by misconfiguring the paraid.

This can happen when there are 3 different places you need to update!

This PR makes it so a SINGLE location is the source of truth for the ParaId.

@shawntabrizi
Copy link
Member Author

/command prdoc --audience=runtime dev --bump=minor

@shawntabrizi
Copy link
Member Author

/cmd --audience runtime_dev --bump minor

Copy link

github-actions bot commented Dec 3, 2024

Command "--audience runtime_dev --bump minor" has started 🚀 See logs here

Copy link

github-actions bot commented Dec 3, 2024

Command "--audience runtime_dev --bump minor" has failed ❌! See logs here

@shawntabrizi
Copy link
Member Author

/cmd --audience runtime_dev --bump minor

Copy link

github-actions bot commented Dec 3, 2024

Command "--audience runtime_dev --bump minor" has started 🚀 See logs here

Copy link

github-actions bot commented Dec 3, 2024

Command "--audience runtime_dev --bump minor" has failed ❌! See logs here

Copy link
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

// You MUST set this to the correct network!
para_id: 1000,
},
Extensions { relay_chain: "rococo-local".into(), para_id: runtime::PARACHAIN_ID },
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: can we do the same thing for rococo-local?

@iulianbarbu iulianbarbu added the R0-silent Changes should not be mentioned in any release notes label Dec 4, 2024
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants