widelands

Merge lp://qastaging/~widelands-dev/widelands/relax-ai-asserts into lp://qastaging/widelands

  1. relax-ai-asserts
  2. Merge into trunk
Proposed by GunChleoc
Status: Merged
Merged at revision: 9200
Proposed branch: lp://qastaging/~widelands-dev/widelands/relax-ai-asserts
Merge into: lp://qastaging/widelands
Diff against target: 174 lines (+39/-24)
3 files modified
src/ai/ai_help_structs.cc (+18/-0)
src/ai/ai_help_structs.h (+7/-2)
src/ai/defaultai.cc (+14/-22)
To merge this branch: bzr merge lp://qastaging/~widelands-dev/widelands/relax-ai-asserts
Reviewer Review Type Date Requested Status
TiborB Approve
Review via email: mp+372400@code.qastaging.launchpad.net

Commit message

Some improvements around AI asserts

- Relax AI requirement for hunters
- Be more informative when building is missing the
  "collects_ware_from_map" AI hint

To post a comment you must log in.
Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 5404. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/581765188.
Appveyor build 5174. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_relax_ai_asserts-5174.

Revision history for this message
TiborB (tiborb95) wrote :

code looks OK

review: Approve
Revision history for this message
GunChleoc (gunchleoc) wrote :

Thanks for the review :)

@bunnybot merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: