The -f FILENAMES option is redundant with the input_pdb option, except that the former is optional and allows multiple files, and the latter is mandatory and only allows a single file.
I suggest to deprecate the -f and --file arguments and turn input_pdb into an nargs='+' option.
There is one more difference: input_pdb is only added if build_parser creates a new parser, but not if it creates a subparser. The --file option is always added. I don't know if this is relevant.
The
-f FILENAMESoption is redundant with theinput_pdboption, except that the former is optional and allows multiple files, and the latter is mandatory and only allows a single file.I suggest to deprecate the
-fand--filearguments and turninput_pdbinto annargs='+'option.There is one more difference:
input_pdbis only added ifbuild_parsercreates a new parser, but not if it creates a subparser. The--fileoption is always added. I don't know if this is relevant.