widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/test-ngettext into lp:widelands

To: mp+291587@xxxxxxxxxxxxxxxxxx
From: Klaus Halfmann <klaus.halfmann@xxxxxxxx>
Date: Tue, 12 Apr 2016 13:27:45 -0000
In-reply-to: <20160412074103.5737.86983.launchpad@ackee.canonical.com>
Reply-to: mp+291587@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve compile, regressiontest.py +/-

Having more tests is always good and the code looks OK for me.

test-ngettext klaus$ ./regression_test.py -b ./widelands

OTOH Miroslav question is valid, where or when do we need to show someting like
"You have 3.145972 Item(plurals) in you Inventory"
?
-- 
https://code.launchpad.net/~widelands-dev/widelands/test-ngettext/+merge/291587
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/test-ngettext.

References

[Merge] lp:~widelands-dev/widelands/test-ngettext into lp:widelands
From: GunChleoc, 2016-04-12