Code review comment for lp:~trb143/openlp/audit

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

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])

review: Approve

« Back to merge proposal