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

Revision history for this message
Phill (phill-ridout) wrote :

See my inline comments about your test doc stings and comments.

And also see my inline comments with regard to the Unicode literals. They're not required, and seem only to be implemented to ease porting from py2! https://www.python.org/dev/peps/pep-0414/#proposal

Are there any tests for the new module/method you've addded:
reporting.py
on_tools_report_song_list_triggered

« Back to merge proposal