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

refactor: Umbrella Iroha CLI #5282

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

aoyako
Copy link
Contributor

@aoyako aoyako commented Jan 16, 2025

Related to #4969

This PR begins the unification of cli tools into the iroha/kagami tools.
#4969 (comment)
#4969 (comment)
#4969 (comment)

Changes introduced extend the default scope of kagami by adding:

  • codec (types, json-to-scale, scale-to-json, scale-to-rust)
  • wasm (from wasm_builder)
  • kura (from kura_inspector)
  • swarm (from iroha_swarm)

Documentation in the readme is updated as well. I believe the documentation update on the website will be updated afterward.

The swarm and wasm_builder libraries are left intact in case they are later reused in the code.

Signed-off-by: Lohachov Mykhailo <[email protected]>
Signed-off-by: Lohachov Mykhailo <[email protected]>
Signed-off-by: Lohachov Mykhailo <[email protected]>
Signed-off-by: Lohachov Mykhailo <[email protected]>
Signed-off-by: Lohachov Mykhailo <[email protected]>
Signed-off-by: Lohachov Mykhailo <[email protected]>
@aoyako aoyako marked this pull request as draft January 16, 2025 03:38
@s8sato s8sato self-assigned this Jan 17, 2025
Signed-off-by: Lohachov Mykhailo <[email protected]>
Signed-off-by: Lohachov Mykhailo <[email protected]>
@github-actions github-actions bot added the config-changes Changes in configuration and start up of the Iroha label Jan 17, 2025
Copy link

@BAStos525

@aoyako aoyako added the CLI label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI config-changes Changes in configuration and start up of the Iroha
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants