← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~trb143/openlp/media into lp:openlp

 

> Just a couple things that stood out after a quick peruse.
> 
> 2975 Indentation appears to be out
> 491 - 99, 1440, 1448 Should this be restructured text?
> 1875 You've got an extra line in there.
> 2172 Suffex is spelt incorrectly

This is a diff problem editors (Pycharm and vi show valid indents)

491 - 99 These are object names.

1875 is correct so no change

2172 is fixed and you missed some !!!!
-- 
https://code.launchpad.net/~trb143/openlp/media/+merge/136004
Your team OpenLP Core is subscribed to branch lp:openlp.


References