Skip to content
Snippets Groups Projects

M300433/cli parser fix

Merged Oliver Heidmann requested to merge m300433/cli_parser_fix into develop

With recent changes I tried to remove an error message on an option followed by a operator. This idea backfired in context of -h and --module_info who have operator names as input. As I could not find a way to circumvent the error nor -h not working with what I added I removed the stop parameter which was introduced a few weeks prior. This means that with this merge request a call like cdo -m info -topo Does no longer result in a proper "missing argument" message but with a conversion error that info could not be converted to float. While this behavior is not the optimal solution every other solution I could come up with was not worth it (e.g. Typing arguments).

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading