Review: Approve
Code LGTM, jut 2 questions regarding the comments - see NOCOM in my commit.
Not tested yet.
--
https://code.launchpad.net/~widelands-dev/widelands/ai_colonization_update/+merge/337913
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/ai_colonizat
How about having a weight instead? If building type has no inputs, prefer it if
you can find a suitable spot.
--
https://code.launchpad.net/~widelands-dev/widelands/ai_fisher_identification/+merge/337914
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-de
The claypit/aquafarm relation is exactly the other way around – the clay pit is
basic, the aqua farm should be built only later in the game. The point here is
that building an aqua farm makes no sense at all if there is no (working) clay
pit nearby…
@GunChleoc: Thanks for the proofreading, you
re: "Add a new hint supporter_required..." - yes, this sound good to me, I can
do it and should not be that complicated. Just dont forget about possible
conflicts with basic economy stuff (supported building is a basic, but
supporter is not)
--
https://code.launchpad.net/~widelands-dev/wideland
Continuous integration builds have changed state:
Travis build 3189. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/342516094.
Appveyor build 2998. State: failed. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_fh1_maps_pl
I am afraid that in a next tribe the fishers will have an input - worms or
something like that and we will be reworking this again :(
--
https://code.launchpad.net/~widelands-dev/widelands/ai_fisher_identification/+merge/337914
Your team Widelands Developers is requested to review the proposed me
TiborB has proposed merging
lp:~widelands-dev/widelands/ai_fisher_identification into lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/ai_fisher_identification/+merge/337914
AI tweak - a production
TiborB has proposed merging lp:~widelands-dev/widelands/ai_colonization_update
into lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/ai_colonization_update/+merge/337913
Changes to AI in regard to
Codecheck failure is already in trunk, so it needs to be fixed there
@bunnybot merge force
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1669230-lastbastion-finetune/+merge/337083
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1669230-lastbast
Review: Approve
Code LGTM, not tested.
--
https://code.launchpad.net/~widelands-dev/widelands-website/auto_copyr_year/+merge/337911
Your team Widelands Developers is subscribed to branch lp:widelands-website.
___
Mailing list: https://launchpad.net/~wi
Yep, Travis still has some catching up to do, because Bunnybot had indeed
gotten confused due to problems with Launchpad.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1669230-lastbastion-finetune/+merge/337083
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev
Oh, I forgot that you did a merge. We might be waiting for travis and appveyor.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1669230-lastbastion-finetune/+merge/337083
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1669230-lastbastion-finetune
When may I conclude that bunnybot has, once again, decided not to merge. What
is the problem this time?
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1669230-lastbastion-finetune/+merge/337083
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-166
Continuous integration builds have changed state:
Travis build 3187. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/342514546.
Appveyor build 2996. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_fh1_tutori
I have done some proofreading on the tribe's units. Will do the scenarios some
other time.
Best look at the diff to see if you agree with my changes - there are also some
NOCOM comments in there for discussion.
2 more things:
1. I don't like "(Master) Seamstress" as the name of a building. The
The proposal to merge
lp:~widelands-dev/widelands/bug-1669230-lastbastion-finetune into lp:widelands
has been updated.
Commit Message changed to:
Minor fixes to Last Bastion map. See LP bug 1669230 for details.
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1669
Evgeny Kapun has proposed merging lp:~abacabadabacaba/widelands/bug1749586 into
lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1749586 in widelands: "Fish breeder can't breed fish if the game is
loaded after all fish is caught"
https://bugs.launchp
I have tested Island Hopping now. Smugglers still needs a full test.
--
https://code.launchpad.net/~widelands-dev/widelands/fh1-maps-multiplayer/+merge/337506
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/fh1-maps-multiplayer into lp:widel
I think this is ok.
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1669230-lastbastion-finetune/+merge/337083
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1669230-lastbastion-finetune.
___
kaputtnik has proposed merging
lp:~widelands-dev/widelands-website/auto_copyr_year into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands-website/auto_copyr_year/+merge/337911
Set the last y
This merged and deployed now.
Thanks Trimard for this feature and being that patient with my slow answers :-)
--
https://code.launchpad.net/~trimardio/widelands-website/module_scheduling/+merge/335570
Your team Widelands Developers is subscribed to branch lp:widelands-website.
__
The proposal to merge lp:~trimardio/widelands-website/module_scheduling into
lp:widelands-website has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~trimardio/widelands-website/module_scheduling/+merge/335570
--
Your team Widelands Developers
I seem to remember a forum discussion, some time ago, about some maps
needing small buildings to conquer mountains as large plots were to far
away and resources could not be mined.
--
https://code.launchpad.net/~widelands-dev/widelands/frisians/+merge/336910
Your team Widelands Developers is su
I have left a comment in the bug - I think we need one more small building plot.
If you want to rename the map directory, please use bzr mv.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1669230-lastbastion-finetune/+merge/337083
Your team Widelands Developers is subscribed to branch
The proposal to merge lp:~widelands-dev/widelands/ai_militarysites_prohibited
into lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/ai_militarysites_prohibited/+merge/337822
--
Your team Widelands Develo
Continuous integration builds have changed state:
Travis build 3184. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/342512052.
Appveyor build 2993. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_ai_militar
Continuous integration builds have changed state:
Travis build 3183. State: failed. Details:
https://travis-ci.org/widelands/widelands/builds/342511097.
Appveyor build 2992. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_string_fix
Continuous integration builds have changed state:
Travis build 3182. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/342509354.
Appveyor build 2991. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_174014
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/ai_militarysites_prohibited/+merge/337822
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/ai_militarysites_prohibited.
___
Mailing list: https://l
29 matches
Mail list logo