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

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

Looks fine. Just 2 things though.

Firstly, coding standards - but I understand that this is "legacy" code, so that's fine.

Secondly, if you're working on the songs plugin, rather create a new branch off trunk with an appropriate name (like "songsplugin"). It's the reason we went with Bazaar :-)

review: Approve

« Back to merge proposal