← Back to team overview

openlp-core team mailing list archive

Thread
Date

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

To: mp+141495@xxxxxxxxxxxxxxxxxx
From: Jonathan Corwin <j@xxxxxxxxxxxx>
Date: Wed, 02 Jan 2013 21:06:22 -0000
In-reply-to: <20121230221322.10395.42595.launchpad@ackee.canonical.com>
Reply-to: mp+141495@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

124, 138 - missing spaces after commas

In the tests, could there be some consistency as to whether there is a space or not between the # and the GIVEN/WHEN/THEN? http://wiki.openlp.org/Development:Unit_Tests has a space.
-- 
https://code.launchpad.net/~trb143/openlp/media/+merge/141495
Your team OpenLP Core is subscribed to branch lp:openlp.

References

[Merge] lp:~trb143/openlp/media into lp:openlp
From: Tim Bentley, 2012-12-30