← Back to team overview

openlp-core team mailing list archive

Thread
Date

Re: [Merge] lp:~phill-ridout/openlp/1209515_2.0 into lp:openlp/2.0

To: mp+180471@xxxxxxxxxxxxxxxxxx
From: Raoul Snyman <raoul@xxxxxxxxxxx>
Date: Fri, 16 Aug 2013 20:34:53 -0000
In-reply-to: <20130816052538.8287.69284.launchpad@ackee.canonical.com>
Reply-to: mp+180471@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

Line 140 should be in there. PEP8 says that there should be 2 blank lines between statements on the module level.

Oh, and even if it says you need to review yourself, please don't.
-- 
https://code.launchpad.net/~phill-ridout/openlp/1209515_2.0/+merge/180471
Your team OpenLP Core is subscribed to branch lp:openlp/2.0.

References

[Merge] lp:~phill-ridout/openlp/1209515_2.0 into lp:openlp/2.0
From: Phill, 2013-08-16