I will have to look at this in detail later, my brain's not working properly 
today.

Codecheck still comes up with an error, could you please always run codecheck 
over the complete code base before submitting a merge request? This way, we 
won't have cosmetic changes in the diff in the future.

I also saw that there are commented out lines of code in the diff, they should 
go before a merge.
-- 
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.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to