spdx / ntia-conformance-checker

Check SPDX SBOM for NTIA minimum elements

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Syntax of the short arguments

vargenau opened this issue · comments

On page https://github.com/spdx/ntia-conformance-checker it says:

Usage: ntia-checker [OPTIONS]

Options:
  --file TEXT            The file to be parsed
  --output [print|json]  Output format  [default: print]
  -v, --verbose          Use verbose printing
  --output_path TEXT     Filepath for optionally storing output.
  --help                 Show this message and exit.

Two issues:

  • the short argument -h is missing in this description
  • the short argument -v is not implemented

It gives:

ntia-checker -v
usage: ntia-checker [-h] [--file FILE] [--output {print,json,html,quiet}] [--verbose] [--output_path OUTPUT_PATH] [--version]
                    [--skip-validation]
ntia-checker: error: unrecognized arguments: -v

Thanks, @vargenau! I really appreciate you finding these bugs.

For anyone reading this issue, PRs welcome!

can i work on this @jspeed-meyers ?

i need to update line 26

   parser.add_argument(
        "--verbose",
        action="store_true",
        help="Specify whether output should be verbose",
    )

to

   parser.add_argument(
       "-v",  "--verbose",
        action="store_true",
        help="Specify whether output should be verbose",
    )

@devbysn, please! I've assigned it to you.