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

fixes sheagcraig/yo#26 #27

Open
wants to merge 4 commits into
base: testing
Choose a base branch
from

Conversation

mikemackintosh
Copy link

Fixes #26 which will occur if the Yo binary is not installed in the expected location

@sheagcraig
Copy link
Owner

This looks great, with one comment:

argparse will automatically show the help text if there are no args, so you should be able to remove that function and test.

@sheagcraig
Copy link
Owner

Ah, but I do see that it doesn't show the full help message. I bet this is just an argparse option that I forgot to set. Let me know if you can't figure out how to always show the full help and I'll take a look.

@sheagcraig sheagcraig changed the base branch from master to testing October 5, 2019 13:26
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