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.

Sign up for GitHub

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

Jump to bottom

Split sub-command run to init and run #1345

Closed
yangby-cryptape opened this issue Aug 22, 2023 · 0 comments · Fixed by #1386
Closed

Split sub-command run to init and run #1345

yangby-cryptape opened this issue Aug 22, 2023 · 0 comments · Fixed by #1386
Assignees
@yangby-cryptape
Labels
t:feature

Comments

@yangby-cryptape
Copy link
Collaborator

Contact Details

No response

Propose-a-new-feature

Since #1307, the non-consensus configuration and consensus configuration are detached.

Currently, there is only one sub-command run, when Axon client runs, it has to check whether it's the first run.

  • Yes, then it have to read the configuration to initialize the chain.
  • Otherwise, then it have to check if user-provided configuration is the same as the chain saved in storage.

If split sub-command run to init and run, the above checks can be ignore.

Alternatives you've considered

No response

Anything else?

No response
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yangby-cryptape yangby-cryptape

Labels
t:feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(cli): split sub-command run to init and run axonweb3/axon
1 participant
@yangby-cryptape