Review: Approve

> There was suggestion by SirVer to add a regression test for it, but I see 
> that LUA api for control a ship movement is needed, so this would be separate 
> job - making LUA interface to navigate the ship and prepare a test for this 
> functionality.

there are other tests that control ships via UI scripting already. You could do 
that too (grep for 'click_on_ship' in tests/maps/expedition.wmf). 

otherwise lgtm.
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai-enhancements/+merge/250207
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/ai-enhancements.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to