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