GunChleoc has proposed merging lp:~widelands-dev/widelands/prevent_ai_deadlocks
into lp:widelands/build19.
Commit message:
Added function to starting conditions to make sure that the AI never runs out
of basic building materials in order to prevent deadlocks.
Requested reviews:
Widelands Deve
I strongly disagree. This is a step to slippery slope - once you get there we
might never stop adding some cheatings to help AI. Moreover we already have
cheating mode - Trading post.
As I understand all that is needed is to add more logs for atlanteans when
starting in village mode. Somehow th
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/fs-tile-background/+merge/309673
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/fs-tile-background.
___
Mailing list: https://launchpad.net/~wide
The proposal to merge lp:~widelands-dev/widelands/fs-tile-background into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/fs-tile-background/+merge/309673
--
Your team Widelands Developers is subscribed
I am in favor of merging this into B19 but NOT into trunk. Would that be OK for
you?
--
https://code.launchpad.net/~widelands-dev/widelands/prevent_ai_deadlocks/+merge/309727
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/prevent_ai_deadlo
Continuous integration builds have changed state:
Travis build 1549. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/172218875.
Appveyor build 1390. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_prevent_ai
I think Tibor is right.
Even if there are posts in the forum we have no bugreprot for this issue. And
from my point of view bug reports are the things to go.
The mentioned failure isn't such a show stopper as some users meant it is and
it's quite worth for working on if there is no bug report.
Review: Disapprove
I also strongly oppose merging this to b19.
I've added a bug report here: https://bugs.launchpad.net/widelands/+bug/1638260
Adding some more logs to the warehouse for fortified villages would be my way
to go.
--
https://code.launchpad.net/~widelands-dev/widelands/prevent_ai_
+1 for adding logs. How many do you think we need?
--
https://code.launchpad.net/~widelands-dev/widelands/prevent_ai_deadlocks/+merge/309727
Your team Widelands Developers is subscribed to branch lp:widelands/build19.
___
Mailing list: https://launchpad
Tino has proposed merging lp:~widelands-dev/widelands/innosetup_appid into
lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/innosetup_appid/+merge/309749
This should allow installing different deve
Review: Disapprove
In my memory deadlocking AI was the norm in b17 and very frequent in b18. That
people complain about it happening now in places is a sign that this has
changed - which is a testament to Tibor's work! Great job!
I think this is not such a big deal as people make it out to be.
The proposal to merge lp:~widelands-dev/widelands/prevent_ai_deadlocks into
lp:widelands/build19 has been updated.
Status: Needs review => Rejected
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/prevent_ai_deadlocks/+merge/309727
--
Your team Widelands Developers
Thanks for adding your experience to the issue. I'll be OK with not having it
for Build 19.
@Tibor: Don't feel bad, it's not your fault that nobody noticed until now!
Listen to SirVer's well-deserved praise instead :)
--
https://code.launchpad.net/~widelands-dev/widelands/prevent_ai_deadlocks/+
GunChleoc has proposed merging
lp:~widelands-dev/widelands/fsmenu_fullscreen_2_about into lp:widelands.
Commit message:
The About menu now relayouts itself when fullscreen mode is toggled.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1398733 in widelands: "Fulls
The proposal to merge lp:~notabilis27/widelands/casern into lp:widelands has
been updated.
Description changed to:
Implements a worker queue which allows production buildings to consume
workers as input.
Adds:
- Adds worker queue for production buildings.
- The "inputs" in the lua files for the
Notabilis has proposed merging lp:~notabilis27/widelands/casern into
lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1075562 in widelands: "Implement a casern for soldier recruting"
https://bugs.launchpad.net/widelands/+bug/1075562
For more details,
The proposal to merge lp:~7010622-q/widelands/topple-production-logic-4 into
lp:widelands has been updated.
Commit Message changed to:
Adjusted time values for weapon and armour production sites (all races) which
were changed recently for more efficient use of their schemes.
The new values are
@bunnybot merge
--
https://code.launchpad.net/~7010622-q/widelands/topple-production-logic-4/+merge/309656
Your team Widelands Developers is requested to review the proposed merge of
lp:~7010622-q/widelands/topple-production-logic-4 into lp:widelands.
The proposal to merge lp:~7010622-q/widelands/topple-production-logic-4 into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~7010622-q/widelands/topple-production-logic-4/+merge/309656
--
Your team Widelands Developers is requ
Continuous integration builds have changed state:
Travis build 1552. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/172272098.
Appveyor build 1393. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_innosetup_
Continuous integration builds have changed state:
Travis build 1554. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/172288018.
Appveyor build 1394. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_fsmenu_ful
Continuous integration builds have changed state:
Travis build 1556. State: failed. Details:
https://travis-ci.org/widelands/widelands/builds/172322064.
Appveyor build 1396. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_notabilis27_widelands_casern-1396.
GunChleoc has proposed merging lp:~notabilis27/widelands/bug-message-window
into lp:widelands.
Commit message:
Disable center button at once when message window is opened.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1638409 in widelands: "Pressing "center" butt
23 matches
Mail list logo
