-
Notifications
You must be signed in to change notification settings - Fork 124
Completion state #774
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Completion state #774
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
be96014
Removed need for AutoCompleter's token_start_index member
kmvanbrunt 4b4d723
Removed token_start_index from AutoCompleter's __init__ method
kmvanbrunt 474af6f
Replaced AutoCompleter._positional_completers since an ArgumentParser…
kmvanbrunt 6979766
Accounting for value of preserve_quotes when parsing for argparse tab…
kmvanbrunt 71367fb
Added capability for an argparse-based command's completer/choice fun…
kmvanbrunt bafdcca
Fixed warning
kmvanbrunt 6f5a20d
Combined 2 functions
kmvanbrunt 3442053
Now adding flags with no arguments to consumed_arg_values. This way c…
kmvanbrunt 125eba8
Renamed some variables
kmvanbrunt 90cc598
Merge branch 'master' into completion_state
kmvanbrunt 301fd2c
Minor grammar improvement in CHANGELOG
tleonhardt 422f90a
Fixed comment
kmvanbrunt f1bf0ae
Added parser to parsed_args namespace
kmvanbrunt 6af36b1
AutoCompleter now passes parsed_args to all choices and completer fun…
kmvanbrunt 33adefc
AutoCompleter only passes parsed_args to choices/completer functions …
kmvanbrunt 294d911
Updated comment
kmvanbrunt e58b95f
Merge branch 'master' into completion_state
kmvanbrunt d6643af
Change parsed_args to arg_tokens
kmvanbrunt 8acd840
Merge branch 'completion_state' of github.com:python-cmd2/cmd2 into c…
kmvanbrunt cdc0efa
Updated documentation
kmvanbrunt eda4014
Added unit tests for arg_tokens
kmvanbrunt 95174ff
Updated change log
kmvanbrunt ed56264
Increased code coverage
kmvanbrunt 65ed685
Fixed issue where invalid subcommand token was ignored when tab compl…
kmvanbrunt 9a7818b
Fixed issue where invalid subcommand token was ignored when retrievin…
kmvanbrunt e6585d1
Changed arg_tokens to a dictionary
kmvanbrunt 89fa7fb
Merge branch 'master' into completion_state
kmvanbrunt 68e7528
Updated arg_tokens unit tests
kmvanbrunt 2e94459
Added unit test to override a parameter in a parent parser
kmvanbrunt c7bf16f
Fixed unit tests on Python versions older than 3.7
kmvanbrunt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the change to a dictionary instead of an argparse.Namespace