Closed ColinDKelley closed 3 years ago
Currently the code logs 4 different ways:
Listen.logger.info "message"
Listen::Logger.info "message"
_log :info, "message"
self.class._log :info, "message"
These can all be unified with (1).
Also, there is a comment about clearing the logger, but the code doesn't actually support an empty (nil) logger. Instead, we can just have a comment describing how the caller can point the logger to /dev/null if they don't want logging.
nil
/dev/null
Currently the code logs 4 different ways:
Listen.logger.info "message"
Listen::Logger.info "message"
_log :info, "message"
self.class._log :info, "message"
These can all be unified with (1).
Also, there is a comment about clearing the logger, but the code doesn't actually support an empty (
nil
) logger. Instead, we can just have a comment describing how the caller can point the logger to/dev/null
if they don't want logging.