fixes and minor changes

README.md

@@ -29,8 +29,7 @@ Note that logs may be delayed due to rate limiting, etc. If you are logging *a l
 
 
 ## Customized Child Loggers
-`setup()` has an optional parameter: `modifyChildLoggersFunc`. If it is given, it must be a function that would take in each created child logger
-and return a child logger. This can be used to add additional things like handlers to each child logger.
+`setup()` has an optional parameter: `modifyChildLoggersFunc`. If it is given, it must be a function that would take in as a single arg, each created child logger. This can be used to add additional things like handlers to each child logger. (Note that the child logger is modified in place).
 
 ## Installation
 ```

csmlog/__init__.py

@@ -16,7 +16,7 @@
 from csmlog.udp_handler import UdpHandler
 from csmlog.udp_handler_receiver import UdpHandlerReceiver
 
-__version__ = "0.26.0"
+__version__ = "0.27.0"
 
 DEFAULT_LOG_FORMAT = "%(asctime)s - %(name)s:%(lineno)d - %(levelname)s - %(message)s"
 
@@ -74,7 +74,7 @@ def getLogger(self, name):
  logger.sysCall = LoggedSystemCall(logger)
 
  if self.modifyChildLoggersFunc:
- logger = self.modifyChildLoggersFunc(logger)
+ self.modifyChildLoggersFunc(logger)
 
  return logger
 
@@ -274,6 +274,7 @@ def setup(
  clearLogs=False,
  udpLogging=True,
  googleSheetShareEmail=None,
+ formatter=None,
  modifyChildLoggersFunc=None,
  ):
  """must be called to setup the logger. Passes args to CSMLogger's constructor"""
@@ -289,7 +290,8 @@ def setup(
  clearLogs,
  udpLogging,
  googleSheetShareEmail,
- modifyChildLoggersFunc,
+ formatter=formatter,
+ modifyChildLoggersFunc=modifyChildLoggersFunc,
  )
  self._activeCsmLogger.parentLogger.debug("==== %s is starting ====" % appName)
 

csmlog/tests/test_csmlog.py

@@ -231,7 +231,6 @@ def test_formatter_setting(csmlog):
 def test_modify_child_loggers_func(csmlog):
  def _tmp(logger):
  logger.lolcats = "lolcats"
- return logger
 
  csmlog.modifyChildLoggersFunc = _tmp