Code review comment for lp:~victor-pelt/widelands/workers

Revision history for this message
SirVer (sirver) wrote :

Looks good so far. Just two things: there are text conflicts against trunk, there will likely be some more when Lua is merged. Second: what does the worker program do? Is this to automatically evict workers? Can you provide a proof of concept enhancement to eg. one of the mines?

review: Needs Fixing

« Back to merge proposal