← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1366725 into lp:widelands

 

> SirVer is too busy - generally not just now. I communicated with him and he suggested that also other developers should review/approve merge requests. (I had my AI branch in mind specifically)
> 
> So I can approve this one as this looks a simple one, and SirVer has looked at it before, and I tested it. Would you mind?

OK, I'll merge this then. Thanks for the review.

I'd rather not review the AI branch for a merge, because that's over my
head unfortunately. I am quite OK with general architecture and
consistency of stuff, but the ins and outs of heuristics and the finer
points of C++ are beyond me.

-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1366725/+merge/233750
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1366725.


References