widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/document-metaserver-option into lp:widelands

To: mp+343841@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Tue, 24 Apr 2018 07:40:23 -0000
In-reply-to: <152450738071.12942.16456218461242347207.launchpad@ackee.canonical.com>
Reply-to: mp+343841@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

I don't understand your TODO comment. Not all of them are available in the Options screen, but they should all do something. Also, ai_training is documented, so which ones do you mean?

I remember we had a bug somewhere about guaranteeing completeness, but I can't find it.

LGTM otherwise :)
-- 
https://code.launchpad.net/~widelands-dev/widelands/document-metaserver-option/+merge/343841
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/document-metaserver-option.

References

[Merge] lp:~widelands-dev/widelands/document-metaserver-option into lp:widelands
From: Notabilis, 2018-04-23