-
Notifications
You must be signed in to change notification settings - Fork 2
initial working cli #46
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
Codecov Report❌ Patch coverage is 🚀 New features to boost your workflow:
|
|
@orbeckst I did some initial work on this. |
orbeckst
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way to keep the individual sub-parsers in each module so that the cli module just builds the full parser by calling functions in the sub module?
I'd avoid subprocess.run and instead create a main(...) function in each module that could be called from cli... or something like that.
In this way, all necessary information stays inside each module and if you change something in the module, you don't have to crosscheck cli.py.
|
@rsexton2 I enabled codecov-ai for this repo as requested. |
Fixes #20
Changes made in this Pull Request:
PR Checklist