Skip to content
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

Fix auto-completion #201

Closed
wants to merge 5 commits into from
Closed

Conversation

gcox
Copy link
Contributor

@gcox gcox commented Dec 2, 2020

#200

Auto completion appears to have been broken since v2.x was released since Program doesn't have any special handling for the completion command that the shell scripts invoke. Additionally, the complete function doesn't handle the case of auto completing top level commands when no partial command has been entered.

This PR will correct these issues, starting with tests that demonstrate the problems.

adds test for auto completing command names when no partial name has been entered
adds test to ensure that the 'completion' command, which is invoked by tabtab, is handled
fixes test for completion of command names when no partial command name has been entered.
@gcox
Copy link
Contributor Author

gcox commented Dec 2, 2020

After getting into this, the changes required here are more significant than I imagined and are probably best made by @mattallty.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant