openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #19350
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