openlp-core team mailing list archive

[Tim Bentley <tim.bentley@xxxxxxxxx>]

Why do you have a def on the front of each line.  That is for functions and
pep thinks you are defining functions not lines of code.
remove the def and try again.

On 19 April 2016 at 20:54, Chris Hill <chris@xxxxxxxxxxxx> wrote:

> I think fixing this will involve putting a blank line before every 'def',
> and splitting the 'def' onto two lines.
>
> I can do this, but think the blank line looks very ugly...
> --
> https://code.launchpad.net/~minkus/openlp/naturalsortfix/+merge/292101
> Your team OpenLP Core is subscribed to branch lp:openlp.
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openlp-core
> Post to     : openlp-core@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openlp-core
> More help   : https://help.launchpad.net/ListHelp
>



-- 
Tim and Alison Bentley
Home@xxxxxxxxxxxxxxx

https://code.launchpad.net/~minkus/openlp/naturalsortfix/+merge/292101
Your team OpenLP Core is subscribed to branch lp:openlp.