search

nasa / fpp

F Prime Prime: A modeling language for F Prime
https://fprime.jpl.nasa.gov
Apache License 2.0
49 stars 31 forks source link

Revise handling of incorrect options to FPP tools #294

Closed bocchino closed 1 year ago

bocchino commented 1 year ago

Ensure that the tools return nonzero status when the command-line options are invalid. Previously, if the option parser returned a None value indicating a parse failure, the tools would exit with status zero.

LeStarch commented 1 year ago

Ohhhhh I have studied and can review this now!

bocchino commented 1 year ago

Thanks, these are good points! I will address them.

bocchino commented 1 year ago

I factored the boilerplate code for the main method into a function in util.Tool. This avoids repeating the boilerplate in each tool.