wrapping python

[leej3]

https://github.com/vanessa-leone/hm_analysis_tool/blob/40bf397c86d076cb3c2ebdc67e66da08d0fa81cd/hm_analysis_tool/wrapper_example.sh#L1

Ideally you should use argparser or click cli or some other python package to generate a command line interface. no need to use bash here

[leej3]

More specifically, take a look at this: https://realpython.com/python-main-function/

[vanessa-leone]

Thanks for you suggestions. I have added a main function to execute only if I am running from the command line. I have defined the variables to run the main function, so I can do testing. I hope that now is better. I have also cleaned the directory and added installation instructions in the README.md I have pushed all the changes, and hopefully for next push I'll have some tests

…

On Sun, Dec 8, 2019 at 4:23 PM john lee ***@***.***> wrote: https://github.com/vanessa-leone/hm_analysis_tool/blob/40bf397c86d076cb3c2ebdc67e66da08d0fa81cd/hm_analysis_tool/wrapper_example.sh#L1 Ideally you should use argparser or click cli or some other python package to generate a command line interface. no need to use bash here — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ANFTMECPBTL5WNZSLYU64X3QXVQTPANCNFSM4JYATKOA> .