-
Notifications
You must be signed in to change notification settings - Fork 4
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
Remove print statements #7
Comments
@EUNIX-TRIX why is it closed before solving it completely? |
Can you tell me what more changes I should do. |
Line 28 |
Sir , but if I remove it then how would user know what option to choose so I left it as it is . |
Logging at info level should should be used for this. All other should be done at debug or errror level. I expected you to make a logger.py where one can define various level, something just like https://github.com/MacBox7/torque/blob/master/raspberry-client/logger.js. |
Ya , sure sir I'll do it. |
A deployment version never has print statements. Make a
logger.py
and print logs by importing it. The purpose is not to make things easy but to make them understand how things actually happen and how to code in the right way. Refer this: https://docs.python.org/2/library/logging.htmlThe text was updated successfully, but these errors were encountered: