Review: Approve code review
Nothing to really check here.
I will it compile it anyway
--
https://code.launchpad.net/~widelands-dev/widelands/compiler-warnings-201908/+merge/372109
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/compiler-warnings-201908.
_
Review: Approve
Compiled withouth any noteworthy complaints
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/compiler-warnings-201908/+merge/372109
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/compiler-warnings-201908.
__
Thanks for the review!
I'll be going through the Travis/Appveyor logs to see if I can find more stuff
to fix.
--
https://code.launchpad.net/~widelands-dev/widelands/compiler-warnings-201908/+merge/372109
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/compiler
Review: Approve
Thanks!
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/ai_wai_files_29082019/+merge/372083
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/ai_wai_files_29082019.
___
Mailing l
LGTM :)
Just 1 tiny nit: What does ga mean? Add a comment to the test file.
--
https://code.launchpad.net/~widelands-dev/widelands/ai_ga_unittests/+merge/372091
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/ai_ga_unittests into lp:widelan
Savegame compatibility hack is up :)
Thanks for the review and codestyle.
--
https://code.launchpad.net/~widelands-dev/widelands/ferry/+merge/351880
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/ferry.
___
Mailing
GunChleoc has proposed merging
lp:~widelands-dev/widelands/compiler-warnings-201908-2 into lp:widelands.
Commit message:
Fix compiler warnings and refactor config dir.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~widelands-dev/wide
I have pushed a commit with a comment.
--
https://code.launchpad.net/~widelands-dev/widelands/ferry/+merge/351880
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/ferry.
___
Mailing list: https://launchpad.net/~widelan
Continuous integration builds have changed state:
Travis build 5365. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/579080055.
Appveyor build 5135. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_compiler_w
The proposal to merge lp:~widelands-dev/widelands/compiler-warnings-201908 into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/compiler-warnings-201908/+merge/372109
--
Your team Widelands Developers i
Continuous integration builds have changed state:
Travis build 5362. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/578917768.
Appveyor build 5132. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_ai_wai_fil
The proposal to merge lp:~widelands-dev/widelands/ai_wai_files_29082019 into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/ai_wai_files_29082019/+merge/372083
--
Your team Widelands Developers is subs
Continuous integration builds have changed state:
Travis build 5366. State: canceled. Details:
https://travis-ci.org/widelands/widelands/builds/579080616.
Appveyor build 5136. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_spritesh
Continuous integration builds have changed state:
Travis build 5367. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/579089392.
Appveyor build 5137. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_794407
GunChleoc has proposed merging
lp:~widelands-dev/widelands/fix-cmakelists-codecheck into lp:widelands.
Commit message:
Dependency check doesn't work with ninja, so we run it manually from compile.sh
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.la
ga = Genetic Algorithm :)
I am going to add the comment..
--
https://code.launchpad.net/~widelands-dev/widelands/ai_ga_unittests/+merge/372091
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/ai_ga_unittests into lp:widelands.
__
Review: Approve
Thanks!
I have done some final cleanup, so this can go in now :)
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/ai_ga_unittests/+merge/372091
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/ai_ga_unittests.
___
Continuous integration builds have changed state:
Travis build 5372. State: failed. Details:
https://travis-ci.org/widelands/widelands/builds/579190053.
Appveyor build 5142. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_spriteshee
Continuous integration builds have changed state:
Travis build 5373. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/579215280.
Appveyor build 5143. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_fix_cmakel
Review: Resubmit
I'm done checking the logs, so this is ready for review now.
--
https://code.launchpad.net/~widelands-dev/widelands/compiler-warnings-201908-2/+merge/372111
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/compiler-warnings-201908-2.
__
20 matches
Mail list logo
