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

feat: ignite chain genesis config option 2.0 #3056

Merged
merged 18 commits into from
Nov 7, 2022
Merged

Conversation

aljo242
Copy link
Contributor

@aljo242 aljo242 commented Nov 3, 2022

Closes #2852

Example:

ignite network chain publish github.com/aljo242/test --genesis-config ./docs/config_test.yml
Source code fetched
Blockchain set up
Chain's binary built
Blockchain initialized
Genesis initialized
✔ Network published 
⋆ Launch ID: 41 

Adds logic to chainconfig for a specific networkconfig. This config only includes the Accounts and Genesis fields, and has custom validation logic.

@lumtis
Copy link
Contributor

lumtis commented Nov 4, 2022

What's the context of the second PR?

Is it easier to redo in order to address the double initialization refactoring?

@aljo242
Copy link
Contributor Author

aljo242 commented Nov 4, 2022

Yep - I'll just close the other one

@aljo242 aljo242 marked this pull request as ready for review November 4, 2022 12:53
@aljo242
Copy link
Contributor Author

aljo242 commented Nov 4, 2022

@lubtd ready for review. Works for me finally

tbruyelle
tbruyelle previously approved these changes Nov 4, 2022
Copy link
Contributor

@tbruyelle tbruyelle left a comment

Choose a reason for hiding this comment

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

lgtm

@lumtis
Copy link
Contributor

lumtis commented Nov 4, 2022

Give me the time today to test it before merging, thanks

Copy link
Contributor

@lumtis lumtis left a comment

Choose a reason for hiding this comment

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

LGTM
need to test the command

@lumtis
Copy link
Contributor

lumtis commented Nov 4, 2022

Tested

Everything is good from my side except the few comments👍

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>
@aljo242 aljo242 requested review from lumtis and tbruyelle November 7, 2022 12:29
Copy link
Contributor

@lumtis lumtis left a comment

Choose a reason for hiding this comment

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

🎉

@lumtis lumtis merged commit db60d71 into develop Nov 7, 2022
@lumtis lumtis deleted the feat/config-gen-redo branch November 7, 2022 14:17
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
* add options

* fix test

* gen mocks

* test

* network config

* account validation

* expand validation rules

* finalize genesis init

* changelog

* Update parser.go

* Update init.go

* add integration test

* Update ignite/chainconfig/parser.go

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>

* remove TODO

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

network: support Ignite CLI config as initial genesis
3 participants