Hi,
I saw this morning those reviewed but not merged code changes in gerrit:
Parser issue for HTML definition list Bug 11748: Handle optionally-closed HTML tags without tidy 2012-04-17 Owner: GWicke Review: +1 by saper https://bugzilla.wikimedia.org/11748 https://gerrit.wikimedia.org/r/#/c/5174/ (bug 32381) Allow descending order for list=backlinks, list=embeddedin and list=imageusage 2012-04-30 Owner : Umherirrender Review: +1 by Aaron Schulz https://bugzilla.wikimedia.org/32381 https://gerrit.wikimedia.org/r/#/c/6108/ Upgrade cortado-ovt to newer version (seems to work fine locally) 2012-05-05 Owner : Reedy Review : +1 by awjrichards https://gerrit.wikimedia.org/r/#/c/6640/
Would it be interesting to generate an automated report detecting > 45 jours code submission having one at least +1 review but still not merged?
The analytics team has written A script to generate such reports and we will publish these results shortly once we have enough data points Best
Diederik
Sent from my iPhone
On 2012-06-14, at 4:31, Sébastien Santoro dereckson@espace-win.org wrote:
Hi,
I saw this morning those reviewed but not merged code changes in gerrit:
Parser issue for HTML definition list Bug 11748: Handle optionally-closed HTML tags without tidy 2012-04-17 Owner: GWicke Review: +1 by saper https://bugzilla.wikimedia.org/11748 https://gerrit.wikimedia.org/r/#/c/5174/
(bug 32381) Allow descending order for list=backlinks, list=embeddedin and list=imageusage 2012-04-30 Owner : Umherirrender Review: +1 by Aaron Schulz https://bugzilla.wikimedia.org/32381 https://gerrit.wikimedia.org/r/#/c/6108/
Upgrade cortado-ovt to newer version (seems to work fine locally) 2012-05-05 Owner : Reedy Review : +1 by awjrichards https://gerrit.wikimedia.org/r/#/c/6640/
Would it be interesting to generate an automated report detecting > 45 jours code submission having one at least +1 review but still not merged?
-- Best Regards, Sébastien Santoro aka Dereckson http://www.dereckson.be/
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
Sébastien Santoro dereckson@espace-win.org wrote:
Hi,
I saw this morning those reviewed but not merged code changes in gerrit:
Parser issue for HTML definition list Bug 11748: Handle optionally-closed HTML tags without tidy 2012-04-17 Owner: GWicke Review: +1 by saper https://bugzilla.wikimedia.org/11748 https://gerrit.wikimedia.org/r/#/c/5174/
I have just rebased this one to the current master.
https://gerrit.wikimedia.org/r/#/c/5174/
git fetch https://gerrit.wikimedia.org/r/mediawiki/core refs/changes/74/5174/3 && git checkout FETCH_HEAD
//Saper
wikitech-l@lists.wikimedia.org