CLI improvements
- Remove clap relics
Better input, help, and print CLI interface:
- Typo
- Better English
- …
Merge request reports
Activity
added 5 commits
-
2128f5ea...4e902403 - 2 commits from branch
dev
- 4a3f79a7 - [mod] remove old clap CLI tool usage relics
- bd762e3a - [mod] improve command line interface
- 8e0bfca0 - [mod] improve keys sub-command interface
Toggle commit list-
2128f5ea...4e902403 - 2 commits from branch
@moul Thank you very much for this contribution:)
I just checked the changes and everything seems okay.
You just need to rename your commit according to the project conventions and then I will merge.
I rebased on
dev
branch (push --force) and the piplines results are not displayed here.Yes it is normal, the CI only starts when the changes concern the rust code, but the 2 commits that you have to recover by making a rebase have not modified the rust code.
Edited by ÉloïsWe always use clap in reality, StructOpt is based on clap.
StructOpt is much cleaner because it allows you to directly obtain a Rust structure containing all the information of the command entered by the user.
Also StructOpt is becoming a standard in the Rust ecosystem, most cli programs now use StructOpt.
Edited by ÉloïsThere is an incoming MR to add new
[enh]
and[mod]
types.We do not use the same commit naming conventions in durs project,
[enh]
and[mod]
don't exist. Please look at the project conventions : https://git.duniter.org/nodes/rust/duniter-rs/blob/dev/doc/fr/developpeurs/conventions-git.md#liste-des-types-de-commitEdited by Éloïsadded 7 commits
-
e9a26273...5fd550fc - 4 commits from branch
dev
- 12aea3cb - [ref] core: remove old clap CLI tool usage relics
- 9f15159b - [ref] core: improve command line interface
- 44eda1c8 - [ref] core: improve keys sub-command interface
Toggle commit list-
e9a26273...5fd550fc - 4 commits from branch
Review: ok
Thank you very much for this excellent contribution :)
Edited by Éloïsenabled an automatic merge when the pipeline for 44eda1c8 succeeds