widelands-dev team mailing list archive

Thread
Date

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

To: mp+242271@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Wed, 03 Dec 2014 08:53:32 -0000
In-reply-to: <20141203083010.26439.56617.codereview@gac.canonical.com>
Reply-to: mp+242271@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

> No more big branches, this is why I enjoyed these two (or three) small
> branches I did lately (including this one and tha LUA resource think - though
> this gone wild from one-line change).

This has happened to me too and will keep happening - one day I will learn that there are no quick fixes. I sometimes find it frustrating at first as well, but when the branch is finished, I am happy, because the code is better.
-- 
https://code.launchpad.net/~widelands-dev/widelands/seafaring-ai/+merge/242271
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/seafaring-ai into lp:widelands.

References

Re: [Merge] lp:~widelands-dev/widelands/seafaring-ai into lp:widelands
From: TiborB, 2014-12-03