search

jessehamner / WeatherWidget

Python and Shell scripts to get radar and hazardous weather outlook info, plus local conditions, from the NWS API. New RIDGE II radar compatibility coming soon.
Apache License 2.0
5 stars 0 forks source link

Convert script and modules to use logging functionality. #8

Closed jessehamner closed 3 years ago

jessehamner commented 3 years ago

print statements have got to go, except for a small subset of use cases.

jessehamner commented 3 years ago

Underway. Have converted alerts, hwo, and current_conditions to logging.

jessehamner commented 3 years ago

Several more files converted over, plus better logging in a few files.

jessehamner commented 3 years ago

It has been useful, while tracking down other problems, to add debug logging and improve the logging as I work on issues. This issue (and working on this issue) provides a good example of the value of documentation and logging.

jessehamner commented 3 years ago

I think I've converted every print() statement that should be converted, to logging.whatever().