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 when commands does not exist on command (e.g. it's a multicommand) #34

Closed
wants to merge 1 commit into from

Conversation

davidc
Copy link

@davidc davidc commented Sep 3, 2017

No description provided.

@untitaker
Copy link
Collaborator

untitaker commented Sep 4, 2017

Hmm. What's the point of having a repl if you can only invoke one command? Also how would you add the repl command if you don't have a multicommand?

@davidc
Copy link
Author

davidc commented Sep 4, 2017

It has many commands, they just aren't in a 'commands' dictionary, MultiCommand doesn't have such a field. See for example http://click.pocoo.org/5/commands/#custom-multi-commands

And I don't have a 'repl' command either, I enter repl automatically if no command line arguments are given.

@untitaker
Copy link
Collaborator

The proper fix is to use list_commands as in #39

@untitaker untitaker closed this May 23, 2018
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.

2 participants