Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1746481 into lp:widelands
I probably know what is going on. The problem is a gap when a building exists and is not occupied yet (scratched from the list) AI when considering new buildings tests: - it is on the list of (missing) basic buildings - none is in construction I have to rethink this -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_portspaces into lp:widelands
I believed the name is the person who should improve the code in the future. it is removed now :) -- https://code.launchpad.net/~widelands-dev/widelands/ai_portspaces/+merge/335785 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_portspaces. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/ai_portspaces into lp:widelands
Yep :) No need for the extra ? though, it's supposed to be TODO(tiborb): The reason for the username is that we will know who created the comment, in case somebody else will want to fix it later and has a question about the comment. -- https://code.launchpad.net/~widelands-dev/widelands/ai_portspaces/+merge/335785 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_portspaces. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp