Closed achilleas-k closed 5 years ago
I intend to add automatic checks on pull requests for these, but for now, here's a list of some of the code style issues that need to be addressed:
os.system()
subprocess.call()
check_output()
os.path.join()
"""string"""
# comment
Roger. I've set up pylint and pep8. Will follow this from now on. All instances of os.system() have been removed. You can test it now.
Done.
I intend to add automatic checks on pull requests for these, but for now, here's a list of some of the code style issues that need to be addressed:
os.system()
should never be used. All instances should be replaced bysubprocess.call()
orcheck_output()
.os.path.join()
."""string"""
but a# comment
.