← Back to team overview

widelands-dev team mailing list archive

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?
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/cleanup-statistics-plotter into lp:widelands.