Skip to content

Commit 44cf057

Browse files
committed
remote/client: get_parser() skip some details when building doc
Hide intentionally undocumented subcommands and overlong lists of choices. Signed-off-by: Jonas Rebmann <[email protected]>
1 parent 1d4a043 commit 44cf057

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

labgrid/remote/client.py

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1704,7 +1704,7 @@ class ExportFormat(enum.Enum):
17041704
def __str__(self):
17051705
return self.value
17061706

1707-
def get_parser() -> argparse.ArgumentParser:
1707+
def get_parser(include_undocumented=False) -> argparse.ArgumentParser:
17081708
parser = argparse.ArgumentParser()
17091709
parser.add_argument(
17101710
"-x",
@@ -1745,11 +1745,12 @@ def get_parser() -> argparse.ArgumentParser:
17451745
metavar="COMMAND",
17461746
)
17471747

1748-
subparser = subparsers.add_parser("help")
1748+
if include_undocumented:
1749+
subparser = subparsers.add_parser("help")
17491750

1750-
subparser = subparsers.add_parser("complete")
1751-
subparser.add_argument("type", choices=["resources", "places", "matches", "match-names"])
1752-
subparser.set_defaults(func=ClientSession.complete)
1751+
subparser = subparsers.add_parser("complete")
1752+
subparser.add_argument("type", choices=["resources", "places", "matches", "match-names"])
1753+
subparser.set_defaults(func=ClientSession.complete)
17531754

17541755
subparser = subparsers.add_parser("monitor", help="monitor events from the coordinator")
17551756
subparser.set_defaults(func=ClientSession.do_monitor)
@@ -1998,7 +1999,7 @@ def get_parser() -> argparse.ArgumentParser:
19981999
"-p",
19992000
"--partition",
20002001
type=int,
2001-
choices=range(0, 256),
2002+
choices=(range(0, 256) if include_undocumented else None),
20022003
metavar="0-255",
20032004
default=1,
20042005
help="partition number to mount or 0 to mount whole disk (default: %(default)s)",
@@ -2091,7 +2092,7 @@ def main():
20912092
initial_state = os.environ.get("LG_INITIAL_STATE", None)
20922093
token = os.environ.get("LG_TOKEN", None)
20932094

2094-
parser = get_parser()
2095+
parser = get_parser(include_undocumented=True)
20952096

20962097
# make any leftover arguments available for some commands
20972098
args, leftover = parser.parse_known_args()

0 commit comments

Comments
 (0)