Skip to content

Commit 43ce775

Browse files
authored
Merge pull request #141 from vladak/enable_logger_nits
use correct terminology for enable_logger()
2 parents 0e5bb0c + 9d66d9f commit 43ce775

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

adafruit_minimqtt/adafruit_minimqtt.py

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1074,16 +1074,22 @@ def is_connected(self):
10741074
return self._is_connected and self._sock is not None
10751075

10761076
# Logging
1077-
def enable_logger(self, logger, log_level=20):
1078-
"""Enables library logging provided a logger object.
1077+
def enable_logger(self, log_pkg, log_level=20, logger_name="log"):
1078+
"""Enables library logging by getting logger from the specified logging package
1079+
and setting its log level.
10791080
1080-
:param logger: A python logger pacakge.
1081+
:param log_pkg: A Python logging package.
10811082
:param log_level: Numeric value of a logging level, defaults to INFO.
1083+
:param logger_name: name of the logger, defaults to "log".
1084+
1085+
:return logger object
10821086
10831087
"""
1084-
self.logger = logger.getLogger("log")
1088+
self.logger = log_pkg.getLogger(logger_name)
10851089
self.logger.setLevel(log_level)
10861090

1091+
return self.logger
1092+
10871093
def disable_logger(self):
10881094
"""Disables logging."""
10891095
if not self.logger:

0 commit comments

Comments
 (0)