Review: Needs Fixing Line 17, 26: need a space after the comma
range(0,len(VerseType.names)) => range(0, len(VerseType.names)) Line 76: Your comment doesn't match what you're actually testing: # THEN: The result should be VerseType.Other self.assertIsNone(... -- https://code.launchpad.net/~strada/openlp/bug-1312629/+merge/217295 Your team OpenLP Core is subscribed to branch lp:openlp. _______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : [email protected] Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp
