widelands-dev team mailing list archive

Thread
Date

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

To: mp+336910@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Tue, 27 Feb 2018 08:06:55 -0000
In-reply-to: <151739936062.8475.1878337614523939807.launchpad@ackee.canonical.com>
Reply-to: mp+336910@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

I have done some proofreading on the first scenario now - feel free to revert or change anything that you don't agree with.

Please have a look at my NOCOM comments in the commit, and also at the handling of Title Case.

Once scenario 2 has been fixed up (see my code style comments above too), I will continue the review and proofreading.
-- 
https://code.launchpad.net/~widelands-dev/widelands/frisians/+merge/336910
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/frisians.

References

[Merge] lp:~widelands-dev/widelands/frisians into lp:widelands
From: Benedikt Straub, 2018-01-31