← Back to team overview

openlp-core team mailing list archive

  1. openlp-core team
  2. Mailing list archive
  3. Message #00534

Re: [Merge] lp:~trb143/openlp/audit into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Approve
Looks fine to me.

TODOs: We should look at also changing this to a log.exception

    log.error(u'Failed to import module %s on path %s for reason %s',
        modulename, path, e.args[0])
-- 
https://code.launchpad.net/~trb143/openlp/audit/+merge/13173
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next