-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Move cli.py to __main__.py to allow for python -m package
#481
base: master
Are you sure you want to change the base?
Conversation
Nice to see you working on it. I'll check on it later. |
I thought we could do something like flask, just add one |
Hmm. I think my intuition was that it's simpler to just have
Also, it doesn't seem like that big of a deal to just add one more file, especially considering how many files tend to end up in a python package project Would you like me to refactor this PR to use an approach more like flask's? |
Yeah, I think that's better. I'd be happy to merge it if you refactor it. |
Hello! I love this template. While using it at a pycon tutorial, I noticed that it doesn't have a
__main__.py
, so it's not possible to runpython -m package_name
. I thought I'd submit a PR to make that possible.Let me know what i'm missing, or if this is totally off-base. And thanks again for the awesome template :)