← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~minkus/openlp/naturalsortfix into lp:openlp

 

I have def because Raoul asked me to extract the lambdas into the line above for performance reasons
Doing this works but fails PEP8: 'E731 do not assign a lambda expression, use a def'

Using a def also works but fails E301 and E704 unless I put a blank line before each 'def' and split each onto two lines.

Removing the def doesn't work (need to define or create a lambda function for use with sort())

*frustration*


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


References