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.
_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help : https://help.launchpad.net/ListHelp
