Last modified: 2014-09-22 22:07:41 UTC
In a similar way that we removed -1 changesets, we should also remove those containing WIP in the summary, since they are not intended to be resolved under the same time pressure. https://gerrit.wikimedia.org/r/#/q/message:%22wip%22,n,z
Ok, filter included in: https://github.com/VizGrimoire/GrimoireLib/commit/41db7d572d41de03c40be9a158d9a45fe97b8fa6 All review_time_pending_* metrics now don't include this reviews.
Looks good. Thank you!
I just realized that #11 WikimediaEvents "Waiting for review" at http://korma.wmflabs.org/browser/gerrit_review_queue.html?page=2 is marking 3 changesets for August. However, it should count 1 because the other 2 have "WIP" in the subject: https://gerrit.wikimedia.org/r/#/q/status:open+project:mediawiki/extensions/WikimediaEvents,n,z Meanwhile, "Waiting for review" at http://korma.wmflabs.org/browser/repository.html?repository=gerrit.wikimedia.org_mediawiki_extensions_WikimediaEvents shows zero, which might mean that the WIP changesets are discarded, although zero is not correct either (should be 1) The calculation of time at "Most recent uploads" is correct, though (151 days in August, when the last upload of the one valid changeset was made in April 1st). Linking this report with Bug 70278 because there we are discussing mismatches counting changesets in relation to this metric.
In case it's useful, now apps/android/commons made it to the top of the list, but it has only two WIP open changesets: https://gerrit.wikimedia.org/r/#/q/status:open+project:apps/android/commons,n,z Same with #2 Parsoid https://gerrit.wikimedia.org/r/#/q/status:open+project:mediawiki/extensions/Parsoid,n,z
Quim, WIP reviews are now filtered. And as expected, the two repos above don't appear anymore.
I've checked the top 10 and everything was perfect. Thank you!
