janware/jw-pkg
Watch
1
Code Pull requests Activity Actions

lib.App: Fix --help with required top-level args #65

Merged
Jan Lindemann merged 1 commit from jan/feature/20260817-lib-app-fix-help-with-required-top-level-args into master 2026-08-17 12:10:37 +02:00 AGit
Conversation 0 Commits 1 Files changed 1 +10 -5
1 changed files with 10 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/python/jw/pkg/lib/App.py
View file

@ -229,10 +229,15 @@ class App: # export
)
self._add_arguments(self.__parser)
if not add_all_parsers:
# Parse known args and configure logging, but only if we're not on
# an interactive usage mission. The problem with parse_known_args()
# is that it fails over missing but required arguments, and we
# don't want that to happen and never get to the point where
# '--help' is registered.
args, _ = self.__parser.parse_known_args(argv)
set_log_flags(args.log_flags)
set_log_level(args.log_level)
log(DEBUG, f'-------------- Running: >{pretty_cmd(cmdline)}<')
add_cmds_to_parser(self, self.__parser, self.__cmds, all = add_all_parsers)