← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~patrick-zakweb/openlp/duplicate-song-finder into lp:openlp

 

Review: Needs Fixing

Very difficult to review as resources has removed the line numbers.

Tests look at the structure of tests you are not following existing practices.

Numerous instances of spaces missing fred+bert should be fred + bert have a look at all your code.
-- 
https://code.launchpad.net/~patrick-zakweb/openlp/duplicate-song-finder/+merge/148055
Your team OpenLP Core is subscribed to branch lp:openlp.


References