nfdi4plants/ARCCommander

[Feature Request] Reduce verbosity of error messages (or add argument for the verbosity level)

Closed this issue · 3 comments

Here is a specific example of output that is way to verbose IMO:

arc i person add

  • there is no add subcommand
  • therefore, arccommander tries to find an external executable named add
  • it does not exist, cli tool exits with error code

the output (currently, all of the trext below is printed on typing arc i person add):

  • verbosity level: default, this is the only default output that should be printed here IMO:
    image

    The order seems wrong here though. shouldn't it be:

    Could not parse given commands.
    ERROR: unrecognized argument: 'add'.
    Try checking if executable with given argument name "arc-i-person-add" exists.
    ERROR: No executable, command or script file with given argument name known.
    
  • verbosity level: informational at best, i would suggest to stop printing full help messages on errors per default:

    image

  • verbosity level: debug. This is something an end user should not see per default.

    image

HLWeil commented

There is a verbosity flag: --verbosity (-v) you can use.

With levels from 0 = no info to 2 = full info (standard is 1)

E.g.: arc -v 0 a add -a abc will add the assay with no info.

HLWeil commented

I will close this issue. Feel free to reopen if this isn't the solution you liked.

Is the output i posted still the default output? if so, i would not agree this to be closed, as it is still too verbose