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

Improve generated cli help #581

Merged
merged 2 commits into from
Jul 31, 2023
Merged

Improve generated cli help #581

merged 2 commits into from
Jul 31, 2023

Conversation

preston-evans98
Copy link
Member

Description

Improve the help messages generated by sov-cli.

Before:

$  cargo run generate-transaction accounts update-public-key
error: the following required arguments were not provided:
  <FIELD0>
  <FIELD1>

Usage: main generate-transaction accounts update-public-key <FIELD0> <FIELD1>

For more information, try '--help'.

After:

$ cargo run generate-transaction accounts update-public-key --help
Updates a public key for the corresponding Account. The sender must be in possession of the new key

Usage: main generate-transaction accounts update-public-key <FIELD0> <FIELD1>

Arguments:
  <FIELD0>  The new public key
  <FIELD1>  A valid signature from the new public key

Options:
  -h, --help  Print help

@codecov
Copy link

codecov bot commented Jul 31, 2023

Codecov Report

Merging #581 (e0887f6) into nightly (3835042) will increase coverage by 0.0%.
The diff coverage is 50.0%.

❗ Current head e0887f6 differs from pull request most recent head faa7480. Consider uploading reports for the commit faa7480 to get more accurate results

Files Changed Coverage Δ
...em/module-implementations/sov-accounts/src/call.rs 72.4% <0.0%> (-2.6%) ⬇️
module-system/sov-modules-macros/src/cli_parser.rs 83.3% <100.0%> (+0.1%) ⬆️

... and 6 files with indirect coverage changes

@preston-evans98 preston-evans98 merged commit 168a508 into nightly Jul 31, 2023
@preston-evans98 preston-evans98 deleted the preston/sov-cli branch July 31, 2023 23:42
preston-evans98 added a commit that referenced this pull request Sep 14, 2023
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.

3 participants