Given that in #444 we changed all references of subcommand_name to command_name, we should be consistent and change all references of subcommand_parser to parser. This keeps consistent nomenclature and is more accurate to the organization of code we've implemented via the CLI. While a command like geoips list plugins is technically a child of list, it's its own command, that differs from another list command (ex. geoips list interfaces). This is a nonfunctional command that will likely cause conflicts with downstream PRs so we're leaving this as a TODO.
Requested Update
Description
Given that in #444 we changed all references of
subcommand_nametocommand_name, we should be consistent and change all references ofsubcommand_parsertoparser. This keeps consistent nomenclature and is more accurate to the organization of code we've implemented via the CLI. While a command likegeoips list pluginsis technically a child of list, it's its own command, that differs from another list command (ex.geoips list interfaces). This is a nonfunctional command that will likely cause conflicts with downstream PRs so we're leaving this as a TODO.Background and Motivation
This stems from this comment on #444.
Code to demonstrate issue
Checklist for Completion
subcommand_parserparserinstead