spdx / ntia-conformance-checker

Check SPDX SBOM for NTIA minimum elements
Apache License 2.0
47 stars 18 forks source link

Syntax of the short arguments #155

Closed vargenau closed 9 months ago

vargenau commented 9 months ago

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:

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
jspeed-meyers commented 9 months ago

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

For anyone reading this issue, PRs welcome!

devbysn commented 9 months ago

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",
    )
jspeed-meyers commented 9 months ago

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