Open kmfarley11 opened 3 years ago
The functions in utils.py often implicitly do a sys.exit(1) (_error()...)
sys.exit(1)
This is unkind to those that'd want to use these imports. Default to returning an empty string or false or something, then have the cli detect the return and exit for us...
Should also add a --version in the argparse...
Probably should've named this issue differently, bcoming a refactor bucket. Anyways, consider adding type-hinting to public methods while in here
The functions in utils.py often implicitly do a
sys.exit(1)
(_error()...)This is unkind to those that'd want to use these imports. Default to returning an empty string or false or something, then have the cli detect the return and exit for us...