Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Wikipedia: Wikitech

Some old proposed changes in Gerrit waiting merge, after a code review.

 

 

Wikipedia wikitech RSS feed   Index | Next | Previous | View Threaded


dereckson at espace-win

Jun 14, 2012, 4:31 AM

Post #1 of 3 (74 views)
Permalink
Some old proposed changes in Gerrit waiting merge, after a code review.

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 [at] lists
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


dvanliere at gmail

Jun 14, 2012, 7:45 AM

Post #2 of 3 (69 views)
Permalink
Re: Some old proposed changes in Gerrit waiting merge, after a code review. [In reply to]

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 [at] espace-win> 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 [at] lists
> https://lists.wikimedia.org/mailman/listinfo/wikitech-l

_______________________________________________
Wikitech-l mailing list
Wikitech-l [at] lists
https://lists.wikimedia.org/mailman/listinfo/wikitech-l


saper at saper

Jun 14, 2012, 9:03 AM

Post #3 of 3 (68 views)
Permalink
Re: Some old proposed changes in Gerrit waiting merge, after a code review. [In reply to]

>> Sébastien Santoro <dereckson [at] espace-win> 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 mailing list
Wikitech-l [at] lists
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Wikipedia wikitech RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.