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

2015-06-08 Thread TiborB
Review: Approve I understand that the discussion on forum is over and there are no further objections. I myself tested it and it looks good to me as well. I went over the code, and as far as I can understand it it looks really good. So here is my approve -- https://code.launchpad.net/~widelan

[Widelands-dev] [Merge] lp:~gunchleoc/widelands/bug-1454371 into lp:widelands

2015-06-08 Thread GunChleoc
GunChleoc has proposed merging lp:~gunchleoc/widelands/bug-1454371 into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1454371 in widelands: ""Out of resources" messages should be triggered by productivity" https://bugs.launchpad.net/widelands/+bug

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

2015-06-08 Thread GunChleoc
Yay! Will merge :) -- https://code.launchpad.net/~widelands-dev/widelands/bug-1399621/+merge/258843 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1399621. ___ Mailing list: https://launchpad.net/~widelands-dev P

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1399621 into lp:widelands

2015-06-08 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1399621 into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1399621/+merge/258843 -- Your team Widelands Developers is subscribed to branch lp