Skip to content

Commit 148752f

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 9b26986 commit 148752f

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

labgrid/remote/client.py

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

1707-
def get_parser() -> argparse.ArgumentParser:
1707+
1708+
def get_parser(include_undocumented=False) -> argparse.ArgumentParser:
17081709
parser = argparse.ArgumentParser()
17091710
parser.add_argument(
17101711
"-x",
@@ -1743,11 +1744,12 @@ def get_parser() -> argparse.ArgumentParser:
17431744
metavar="COMMAND",
17441745
)
17451746

1746-
subparser = subparsers.add_parser("help")
1747+
if include_undocumented:
1748+
subparser = subparsers.add_parser("help")
17471749

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

17521754
subparser = subparsers.add_parser("monitor", help="monitor events from the coordinator")
17531755
subparser.set_defaults(func=ClientSession.do_monitor)
@@ -1996,7 +1998,7 @@ def get_parser() -> argparse.ArgumentParser:
19961998
"-p",
19971999
"--partition",
19982000
type=int,
1999-
choices=range(0, 256),
2001+
choices=(range(0, 256) if include_undocumented else None),
20002002
metavar="0-255",
20012003
default=1,
20022004
help="partition number to mount or 0 to mount whole disk (default: %(default)s)",
@@ -2090,7 +2092,7 @@ def main():
20902092
initial_state = os.environ.get("LG_INITIAL_STATE", None)
20912093
token = os.environ.get("LG_TOKEN", None)
20922094

2093-
parser = get_parser()
2095+
parser = get_parser(include_undocumented=True)
20942096

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

0 commit comments

Comments
 (0)