← Back to team overview

widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/bug-1809838-blackroot-target into lp:widelands

To: mp+361290@xxxxxxxxxxxxxxxxxx
From: Toni Förster <toni.foerster@xxxxxxxxx>
Date: Fri, 28 Dec 2018 07:26:10 -0000
In-reply-to: <154592793289.16006.7830122223389285156.launchpad@ackee.canonical.com>
Reply-to: mp+361290@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

LGTM.

The target should stay in I'd say, since it is an second tier ware. And AFAIK currently all second tier wares have economy targets. So why make an exception here?
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1809838-blackroot-target/+merge/361290
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1809838-blackroot-target.

References

[Merge] lp:~widelands-dev/widelands/bug-1809838-blackroot-target into lp:widelands
From: Notabilis, 2018-12-27