-
-
Notifications
You must be signed in to change notification settings - Fork 114
Use Typer instead of argparse #49
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
Conversation
this will simplify addition of subcommands later on.
This reverts commit 513fcc6.
@paketb0te Thanks for this! Just 2 comments for now:
|
@initialcommit-io of course, I just realized that I still have both files for each command in my branch (kept the original ones to look at while modifying the new ones). I moved everything back into the original files now (so that Since we introduced a new dependency on typer, we should probably start including something like a Oh and I removed the obsolete |
@paketb0te Thanks! Setup.py does automatically include the dependencies, so we should be ok for now, but we do still have the other open issue for the dependency system upgrade... |
The PR for #43 :)