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

Refactoring to properly handle Parachain chain-specs. #187

Closed
wants to merge 6 commits into from

Conversation

arturgontijo
Copy link

@arturgontijo arturgontijo commented Jun 3, 2022

Master is currently not working due the fact that Parachains specs are created with the default ID 1000.
So the Parachains 200, 300 and 400 (from config.json) are not producing blocks.

This PR:

  • Outputs Parachains build-spec (plain) in a file.
  • Replaces the default ID 1000 to whatever is in the config.json.
  • Uses the modified plain file to output Parachains build-spec (raw) in a new file.
  • Outputs export-genesis-wasm and export-genesis-state directly to the Relay chain spec.

TODO:

  • Handle existing chain specs files. -> 2799e31
  • Check if chain is a plain spec file (it's considering that it is already the raw one).

@nuke-web3
Copy link

@arturgontijo this it's worth finishing this with #189 coming up in a timely fashion (I presume)?

@arturgontijo
Copy link
Author

Thanks for the heads up @nukemandan.
Closing this in favor of #189

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants