widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #16408
Re: [Merge] lp:~widelands-dev/widelands/cleanup-statistics-plotter into lp:widelands
Since we're down to 1 bug to fix for Build 20, feature freeze needs to be taken very seriously. Any innocent-looking line of code can potentially introduce a breaking bug that would force us to postpone Build 20.
Code LGTM :)
I think having the min value in the statistics from newstats_1 is a good idea, because the bottom value is not 0. Since it bothers hessenfarmer, maybe there is a way to display both?
--
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-plotter into lp:widelands.
References