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

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

@ -229,11 +229,16 @@ class App: # export
) )
self._add_arguments(self.__parser) self._add_arguments(self.__parser)
args, _ = self.__parser.parse_known_args(argv) if not add_all_parsers:
set_log_flags(args.log_flags) # Parse known args and configure logging, but only if we're not on
set_log_level(args.log_level) # an interactive usage mission. The problem with parse_known_args()
# is that it fails over missing but required arguments, and we
log(DEBUG, f'-------------- Running: >{pretty_cmd(cmdline)}<') # 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) add_cmds_to_parser(self, self.__parser, self.__cmds, all = add_all_parsers)