Code review comment for lp:~jtv/launchpad/bug-662552-get-tm-or-dummy

Revision history for this message
Edwin Grubbs (edwin-grubbs) wrote :

Hi Jeroen,

This branch looks good. I just have one comment on an preexisting typo.

-Edwin

>=== modified file 'lib/lp/translations/browser/tests/translationmessage-views.txt'
>--- lib/lp/translations/browser/tests/translationmessage-views.txt 2010-10-18 22:24:59 +0000
>+++ lib/lp/translations/browser/tests/translationmessage-views.txt 2010-10-27 20:18:48 +0000
>@@ -1,4 +1,5 @@
>-= TranslationMessage View =
>+TranslationMessage View
>+=======================
>
> On this section, we are going to test the view class for an
> ITranslationMessage object.
>@@ -11,13 +12,14 @@
> >>> from lp.services.worlddata.interfaces.language import ILanguageSet
> >>> from lp.translations.publisher import TranslationsLayer
>
>-All the tests will be submitted as comming from Kurem, an editor for the POFile
>-that we are going to edit.
>+All the tests will be submitted as comming from Kurem, an editor for the

s/comming/coming/

>+POFile that we are going to edit.
>
> >>> login('<email address hidden>')
>
>
>-== No plural forms ==
>+No plural forms
>+---------------
>
> We are going to see what happens if we get an entry for a language
> without the plural form information.

review: Approve (code)

« Back to merge proposal