Review: Needs Fixing
Please change the AI hint from ts_type=2 to something that is more
understandable outside of the AI context, maybe
trainingssite_type="no_bread_just_meat" or something along these lines. Numbers
are fully opaque to everybody that looks at the conf files.
Diff comments:
>
TiborB has proposed merging lp:~widelands-dev/widelands/ai-military-changes
into lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/ai-military-changes/+merge/253881
Another bunch of AI changes, this
* State: Failed to build
* Recipe: widelands-dev/widelands-daily
* Archive: ~widelands-dev/ubuntu/widelands-daily
* Distroseries: trusty
* Duration: 3 minutes
* Build Log:
https://launchpad.net/~widelands-dev/+archive/ubuntu/widelands-daily/+recipebuild/893128/+files/buildlog.txt.gz
* Uploa
* State: Failed to build
* Recipe: widelands-dev/widelands-daily
* Archive: ~widelands-dev/ubuntu/widelands-daily
* Distroseries: utopic
* Duration: 3 minutes
* Build Log:
https://launchpad.net/~widelands-dev/+archive/ubuntu/widelands-daily/+recipebuild/893129/+files/buildlog.txt.gz
* Uploa
* State: Failed to build
* Recipe: widelands-dev/widelands-daily
* Archive: ~widelands-dev/ubuntu/widelands-daily
* Distroseries: vivid
* Duration: 3 minutes
* Build Log:
https://launchpad.net/~widelands-dev/+archive/ubuntu/widelands-daily/+recipebuild/893130/+files/buildlog.txt.gz
* Upload
I hijacked the code review and asked. Maybe we get some information.
--
https://code.launchpad.net/~hjd/widelands/tests-poc/+merge/250533
Your team Widelands Developers is requested to review the proposed merge of
lp:~hjd/widelands/tests-poc into lp:~widelands-dev/widelands/debian.
_
Could you repush to another branch and suggest that for merging? The diff below
seems screwed up, I think Launchpad cannot keep up.
--
https://code.launchpad.net/~franku/widelands-website/handle_big_images/+merge/247235
Your team Widelands Developers is subscribed to branch lp:widelands-website.
Merged now. Thanks for the changes.
--
https://code.launchpad.net/~franku/widelands-website/forum_notifyings_contain_links_to_post/+merge/253003
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/forum_notifyings_contain_links_to_post into
lp
The proposal to merge
lp:~franku/widelands-website/forum_notifyings_contain_links_to_post into
lp:widelands-website has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~franku/widelands-website/forum_notifyings_contain_links_to_post/+merge/2530
Review: Approve
You, Sir, are awesome! Thanks.
--
https://code.launchpad.net/~franku/widelands-website/changes_to_admin_wlimages/+merge/252996
Your team Widelands Developers is subscribed to branch lp:widelands-website.
___
Mailing list: https://launch
The proposal to merge lp:~franku/widelands-website/changes_to_admin_wlimages
into lp:widelands-website has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~franku/widelands-website/changes_to_admin_wlimages/+merge/252996
--
Your team Widelands
11 matches
Mail list logo