Skip to content

Commit 5e96ee6

Browse files
committed
Since CompletionError handling has been moved from ArgparseCompleter to complete(), there is no longer a need
to suppress tab hints in the top-level parser for do_set.
1 parent 22de858 commit 5e96ee6

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

cmd2/cmd2.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2859,11 +2859,8 @@ def complete_set_value(self, text: str, line: str, begidx: int, endidx: int,
28592859

28602860
# Create the parser for the set command
28612861
set_parser = DEFAULT_ARGUMENT_PARSER(parents=[set_parser_parent])
2862-
2863-
# Suppress tab-completion hints for this field. The completer method is going to create an
2864-
# ArgparseCompleter based on the actual parameter being completed and we only want that hint printing.
28652862
set_parser.add_argument('value', nargs=argparse.OPTIONAL, help='new value for settable',
2866-
completer_method=complete_set_value, suppress_tab_hint=True)
2863+
completer_method=complete_set_value)
28672864

28682865
# Preserve quotes so users can pass in quoted empty strings and flags (e.g. -h) as the value
28692866
@with_argparser(set_parser, preserve_quotes=True)

0 commit comments

Comments
 (0)