The proposal to merge
lp:~widelands-dev/widelands/bug-1791891-key-modifier-inputqueue into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1791891-key-modifier-inputqueue/+merge/354732
--
Your team
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1791891-key-modifier-inputqueue/+merge/354732
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1791891-key-modifier-inputqueue.
___
Mailing
moved the statistics Table definititon to territorial fuctions
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1817550-Statistics-hook-Winconditions/+merge/365086
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/bug-1817550-Statist
The proposal to merge
lp:~widelands-dev/widelands/territorial_unify_notifications into lp:widelands
has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/territorial_unify_notifications/+merge/362502
--
Your team Widelan
Continuous integration builds have changed state:
Travis build 4788. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/524155429.
Appveyor build 4569. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_territoria
Continuous integration builds have changed state:
Travis build 4787. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/524095891.
Appveyor build 4568. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_179189
Review: Approve
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/territorial_unify_notifications/+merge/362502
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/collectors_notification.
___
Mailin
The proposal to merge lp:~widelands-dev/widelands/mipmaps into lp:widelands has
been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/mipmaps/+merge/363780
--
Your team Widelands Developers is subscribed to branch
lp:~widela
Continuous integration builds have changed state:
Travis build 4782. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/523935950.
Appveyor build 4534. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_mipmaps-45
The proposal to merge lp:~widelands-dev/widelands/cleanup-statistics-plotter
into lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/cleanup-statistics-plotter/+merge/365138
--
Your team Widelands Develope
fixed in r8979
--
https://code.launchpad.net/~widelands-dev/widelands/territorial_unify_notifications/+merge/362502
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/collectors_notification.
___
Mailing list: https://la
Transient failure on Travis
@bunnybot merge force
--
https://code.launchpad.net/~widelands-dev/widelands/cleanup-statistics-plotter/+merge/365138
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/cleanup-statistics-plotter.
__
Refusing to merge, since Travis is not green. Use @bunnybot merge force for
merging anyways.
Travis build 4780. State: errored. Details:
https://travis-ci.org/widelands/widelands/builds/523885621.
--
https://code.launchpad.net/~widelands-dev/widelands/cleanup-statistics-plotter/+merge/365138
Yo
Continuous integration builds have changed state:
Travis build 4780. State: errored. Details:
https://travis-ci.org/widelands/widelands/builds/523885621.
Appveyor build 4536. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_cleanup_s
The proposal to merge lp:~widelands-dev/widelands/editor-grid into lp:widelands
has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/editor-grid/+merge/366482
--
Your team Widelands Developers is requested to r
Sorry, this branch is a bit messed up :(
--
https://code.launchpad.net/~widelands-dev/widelands/editor-grid/+merge/366482
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/editor-grid into lp:widelands.
___
Benedikt Straub has proposed merging lp:~widelands-dev/widelands/editor-grid
into lp:widelands with lp:~nordfriese/widelands/workareas as a prerequisite.
Commit message:
Add a grid overlay to the editor, which can be toggled with a button or the
hotkey 'G'
Requested reviews:
Widelands Develop
The proposal to merge lp:~widelands-dev/widelands/editor-grid into lp:widelands
has been updated.
Status: Work in progress => Superseded
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/editor-grid/+merge/366481
--
Your team Widelands Developers is requested to rev
The proposal to merge lp:~widelands-dev/widelands/editor-grid into lp:widelands
has been updated.
Status: Needs review => Work in progress
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/editor-grid/+merge/366481
--
Your team Widelands Developers is requested to r
Benedikt Straub has proposed merging lp:~widelands-dev/widelands/editor-grid
into lp:widelands.
Commit message:
Add a grid overlay to the editor, which can be toggled with a button or the
hotkey 'G'
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1529261 in widela
Tested and working.
I think we should not allow peaceful mode for territorial time, territorial
lord or autocrat, because it can make those game modes unwinnable. UI-wise, you
could hook into the win condition dropdown's selected signal and show/hide the
checkbox.
--
https://code.launchpad.net
I'll have a look.
--
https://code.launchpad.net/~widelands-dev/widelands/territorial_unify_notifications/+merge/362502
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/collectors_notification.
___
Mailing list: https:/
Review: Approve
OK, let's have it then :)
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/cleanup-statistics-plotter/+merge/365138
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/cleanup-statistics-plotter.
_
Yes, introducing new developers - especially ones that are as capable as you
are - definitely does speed up things. Repeatedly spending a lot of time,
energy and frustration on discussions like this one however does not.
And yes, development can always take longer that one has projected, which i
I'd say we merge it as it is now. I'll look into this when I find more time.
--
https://code.launchpad.net/~widelands-dev/widelands/cleanup-statistics-plotter/+merge/365138
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/cleanup-statistics-p
It is funny how you still mention "this instance" when the scale of things here
is whole months. Of course introducing new bugs doesn't speed-up things, but I
don't remember you ever bothering with project's speed to begin with. Still
introducing developers can definitely speed-up things. Especi
Review: Needs Fixing
When the 20 minutes timer reaches 0, the game is not lost or won.
Note that I have pushed to this branch, so don't forget to pull, because that
is so annoying when it happens.
--
https://code.launchpad.net/~widelands-dev/widelands/territorial_unify_notifications/+merge/3625
The proposal to merge lp:~widelands-dev/widelands/bug-1809838-document-mill
into lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1809838-document-mill/+merge/364704
--
Your team Widelands Developers
Do you want to try hour hand at the refactoring that I suggested above before
we merge this?
--
https://code.launchpad.net/~widelands-dev/widelands/cleanup-statistics-plotter/+merge/365138
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/cle
@bunnybot merge force
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1809838-document-mill/+merge/364704
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1809838-document-mill.
___
Mailing list: https:/
Review: Approve
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1809838-document-mill/+merge/364704
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1809838-document-mill.
___
Mailing list: https://lau
The proposal to merge lp:~widelands-dev/widelands/ship_scheduling_2 into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/ship_scheduling_2/+merge/355510
--
Your team Widelands Developers is subscribed t
OK, the double // is correct then :)
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1817550-Statistics-hook-Winconditions/+merge/365086
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/bug-1817550-Statistics-hook-Winconditions int
Considering all the desyncs and unexpected bugs we were struggling with, do you
really think adding a complex new algorithm to the code would have made things
any more predictable, or speeded up the release in any way whatsoever? It would
have potentially slowed things even further. I'm sorry th
You can end a discussion, but you cannot help your case. Quite expectedly, the
"few weeks" of the "release coming shortly" turned out to be more than half a
year (and still counting). Congratulations for the upcoming release, and I wish
it to really happen and not to be "too little too late". I
Thanks for the review.
the double // is intentional to get an integer back (Stats expect integers)
Will check if it is possible to move to territorial functions.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1817550-Statistics-hook-Winconditions/+merge/365086
Your team Widelands Dev
Refusing to merge, since Travis is not green. Use @bunnybot merge force for
merging anyways.
Travis build 4769. State: canceled. Details:
https://travis-ci.org/widelands/widelands/builds/523574258.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1791891-key-modifier-inputqueue/+merge
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/ship_scheduling_2/+merge/355510
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/ship_scheduling_2.
___
Mailing list: https://launchpad.net/~widela
I have pushed a commit with a code review.
I am also getting this error when loading the tribes:
Trying to add unknown battle animation: attack_walk_ne
--
https://code.launchpad.net/~widelands-dev/widelands/per-level-soldier-anims/+merge/354929
Your team Widelands Developers is requested to rev
The proposal to merge lp:~widelands-dev/widelands/editor-resize-map into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/editor-resize-map/+merge/365638
--
Your team Widelands Developers is subscribed t
The proposal to merge lp:~widelands-dev/widelands/fix_fix_dependencies into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/fix_fix_dependencies/+merge/366416
--
Your team Widelands Developers is subscr
41 matches
Mail list logo
