[teampractices] Code review before writing new code

Alex Monk amonk at wikimedia.org
Fri Jun 12 10:44:29 UTC 2015


Is that 'most activity' determined purely by number of patch sets
('uploads')? Because my 48 patch sets on a change since 2 and a half years
ago (a lot of these would've been release notes rebases, etc.) is not quite
the same thing as other people's 62 patchsets in the last year.

On 12 June 2015 at 09:14, Quim Gil <qgil at wikimedia.org> wrote:

> Looking at a patch level, the majority of oldest open patches without any
> review are related to projects deployed to Wikimedia servers indeed:
>
> http://korma.wmflabs.org/browser/scr-backlog.html
>
> The page above also includes a list of open patches with highest activity,
> which means a higher investment of time from uploaders and reviewers, which
> means a bigger risk of wasting Wikimedia donors money with no useful return.
>
> On Thu, Jun 11, 2015 at 3:40 PM, Quim Gil <qgil at wikimedia.org> wrote:
>
>>
>> 12. UploadWizard
>> 16. Wikistats
>> 21. MassMessage
>> 22. Campaigns
>> 30. Echo
>>
>> These names start sounding familiar, right?
>>
>> http://korma.wmflabs.org/browser/gerrit_review_queue.html
>>
>
>
>
> --
> Quim Gil
> Engineering Community Manager @ Wikimedia Foundation
> http://www.mediawiki.org/wiki/User:Qgil
>
> _______________________________________________
> teampractices mailing list
> teampractices at lists.wikimedia.org
> https://lists.wikimedia.org/mailman/listinfo/teampractices
>
>


-- 
Alex Monk
VisualEditor/Editing team
https://wikimediafoundation.org/wiki/User:Krenair_(WMF)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.wikimedia.org/pipermail/teampractices/attachments/20150612/129ae8cc/attachment.html>


More information about the teampractices mailing list