Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[bawolff]

Just FYI, both page forms and Cargo are maintained by Yaron.

--
Brian

On Tue, Sep 10, 2019 at 3:20 AM Andre Klapper 
wrote:

> CR0: Please review and provide guidance if you are familiar with the
> code, and decide (CR±1 or CR±2):
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
> ** Added new feature: order virtual fields by the order they inserted to
> list on cargo store
> ** 2018-December-17
> ** Maintainers/Stewards: ???
>
> *
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508045/
> ** Edit Project Config
> *
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508047/
> ** Adding files
> ** Author is also extension author; author might need to get aware of
>
> https://www.mediawiki.org/wiki/Gerrit/Privilege_policy#Requesting_Gerrit_privileges
> or what's the policy on self-merging these days?
>
> * https://gerrit.wikimedia.org/r/#/c/pywikibot/core/+/508092/
> ** remove dewiki archive template from edit restrictions
> ** 2019-May-05
> ** Maintainers/Stewards: ??? (not sure where Pywikibot lists that)
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/507821/
> ** Modify "SiteExporter.php" and "SiteImporter.php" to handle
> "LanguageCode" property as well
> ** 2019-Jul-14
> ** Maintainers/Stewards: MediaWiki Platform Team
>
> * https://gerrit.wikimedia.org/r/#/c/operations/mediawiki-config/+/508130/
> ** Create new http://www.mediawiki.org/xml/sitelist-1.1/ to reference
> sitelist-1.1.xsd
> ** 2019-May-07
> ** Maintainers/Stewards: MediaWiki Platform Team ?
>
> *
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/509629/
> ** Minimize multiple instance forms when using the Header Tabs extension
> ** 2019-May-13
> ** Maintainers/Stewards: ???
>
> *
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/512642/
> ** Fix autocomplete filter function for no \w chars.
> ** 2019-May-27
> ** Maintainers/Stewards: ???
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Popups/+/527547/
> ** Add popup support for annotated math
> ** 2019-Aug-05
> ** Maintainers/Stewards: WMF Reading Team
>
> * https://gerrit.wikimedia.org/r/#/c/wikidata/query/rdf/+/532373/
> ** PARALLEL_LOADER
> ** 2019-Aug-29
> ** Maintainers/Stewards: ???
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508165/
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508176/
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512628/
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512632/
> * https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510311/
> * https://gerrit.wikimedia.org/r/#/c/wikipeg/+/511820/
> * https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516455/
> * https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516457/
> *
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ParserFunctions/+/511030/
> * https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510070/
> ** Maintainers/Stewards: Parsing Team
> ** See
> https://lists.wikimedia.org/pipermail/wikitech-l/2019-June/092239.html
>
>
>
> CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/429533/
> ** ApiEditPage: Don't swap undo and undoafter parameters
> ** 2019-Jun-21
> ** Maintainers/Stewards: MediaWiki Platform Team
>
>
>
> Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
> you can get notified of new patches in your code areas of interest.
>
> Thanks in advance for your reviews!
>
>
>
> Of last email's 28 listed patches, 4 got merged, 2 got abandoned, 3 got
> CR-1, 1 got CR+1, 1 got V-1.
> Thanks to Yaron, tosfos, Tgr, bblack, Jforrester, Legoktm, tstarling,
> TheDJ, hashar!
>
>
>
> Maintainers/Stewards data taken from
> https://www.mediawiki.org/wiki/Developers/Maintainers
> CR0  source:
> https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
> CR+1
> 
> source:
> https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0
>
> --
> Andre Klapper (he/him) | Bugwrangler / Developer Advocate
> https://blogs.gnome.org/aklapper/
>
>
> ___
> Wikitech-l mailing list
> Wikitech-l@lists.wikimedia.org
> https://lists.wikimedia.org/mailman/listinfo/wikitech-l
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
** Added new feature: order virtual fields by the order they inserted to list 
on cargo store
** 2018-December-17
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508045/
** Edit Project Config
* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508047/
** Adding files
** Author is also extension author; author might need to get aware of
https://www.mediawiki.org/wiki/Gerrit/Privilege_policy#Requesting_Gerrit_privileges
 
or what's the policy on self-merging these days?

* https://gerrit.wikimedia.org/r/#/c/pywikibot/core/+/508092/
** remove dewiki archive template from edit restrictions
** 2019-May-05
** Maintainers/Stewards: ??? (not sure where Pywikibot lists that)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/507821/
** Modify "SiteExporter.php" and "SiteImporter.php" to handle "LanguageCode" 
property as well
** 2019-Jul-14
** Maintainers/Stewards: MediaWiki Platform Team

* https://gerrit.wikimedia.org/r/#/c/operations/mediawiki-config/+/508130/
** Create new http://www.mediawiki.org/xml/sitelist-1.1/ to reference 
sitelist-1.1.xsd
** 2019-May-07
** Maintainers/Stewards: MediaWiki Platform Team ?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/509629/
** Minimize multiple instance forms when using the Header Tabs extension
** 2019-May-13
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/512642/
** Fix autocomplete filter function for no \w chars.
** 2019-May-27
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Popups/+/527547/
** Add popup support for annotated math
** 2019-Aug-05
** Maintainers/Stewards: WMF Reading Team

* https://gerrit.wikimedia.org/r/#/c/wikidata/query/rdf/+/532373/
** PARALLEL_LOADER
** 2019-Aug-29
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508165/
* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508176/
* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512628/
* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512632/
* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510311/
* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/511820/
* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516455/
* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516457/
* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ParserFunctions/+/511030/
* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510070/
** Maintainers/Stewards: Parsing Team
** See https://lists.wikimedia.org/pipermail/wikitech-l/2019-June/092239.html



CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/429533/
** ApiEditPage: Don't swap undo and undoafter parameters
** 2019-Jun-21
** Maintainers/Stewards: MediaWiki Platform Team



Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!



Of last email's 28 listed patches, 4 got merged, 2 got abandoned, 3 got
CR-1, 1 got CR+1, 1 got V-1.
Thanks to Yaron, tosfos, Tgr, bblack, Jforrester, Legoktm, tstarling,
TheDJ, hashar!



Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper (he/him) | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/


___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Subramanya Sastry]

On 6/20/19 2:08 AM, Andre Klapper wrote:


Summary of worst repos with more than one patch waiting:
5x WikiPEG. 4x Parsoid.


Given our current narrow focus on porting Parsoid from JS to PHP, this 
situation will continue for maybe another 2 months since we are not 
merging code on the JS side unless it is part of code cleanup / 
refactoring / fixes to aid the porting.


Tim, C.Scott, and I have engaged with them on Phabricator on a few of 
their many tasks [1] and have clarified on a couple of the tasks that we 
aren't able to engage with this work at this time. After this wikitech-l 
email, I once again mentioned this on gerrit on one of their patches and 
they have acknowledged that they are aware of the hold up. [2]


Any changes we accept to the JS code base will either have to be ported 
to PHP, and besides review, we would have to put that code through QA, 
and deploy it all of which adds to our critical path of porting and 
eventually deploying. Since we are in code freeze mode, any lengthening 
of this path will have impacts on all projects and clients downstream 
since we aren't fixing bugs or supporting other requests for any of them.


The good news is that we are pretty much done with the core porting and 
we are beginning to enter QA and performance testing phase over the next 
quarter and if all goes well, we'll have deployed Parsoid/PHP to 
production by the end of that quarter. At that time, we'll revisit these 
patches and see what is relevant in the PHP landscape.


Thanks for all your patience,

Subbu.

( Parsing Team )

[1] https://phabricator.wikimedia.org/p/Dan1wang/

[2] 
https://gerrit.wikimedia.org/r/c/mediawiki/services/parsoid/+/508165#message-45472b2c188a560f92f3b44f454083638f15ab33



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Greg Grossmeier]

Hi,

On Fri, Jun 21, 2019 at 11:51:10AM +0800, Sam Wilson wrote:
> Is that list meant to contain all extensions that are used on more than 400
> wikis? Because that looks to be 230 at the moment, going on WikiApiary
> numbers [1]. It seems like a hard list to keep up to date, and it doesn't
> add any info that's not already available elsewhere.

It is. For WMF teams the primary section to update is the
https://www.mediawiki.org/wiki/Developers/Maintainers#MediaWiki_extensions_deployed_on_the_Wikimedia_Cluster
section.

> I have a vague memory that there's some tool that reads
> [[Developers/Maintainers]] though, and (maybe?) adds reviewers on Gerrit
> based on it. Am I imagining that?

You're thinking of https://www.mediawiki.org/wiki/Git/Reviewers  :)


Best,

Greg

-- 
| Greg GrossmeierGPG: B2FA 27B1 F7EB D327 6B8E |
| Release Team ManagerA18D 1138 8E47 FAC8 1C7D |

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Sam Wilson]

On 6/20/19 4:24 PM, Andre Klapper wrote:

Thanks for the feedback! Maybe Cargo (et al) should be added under
https://www.mediawiki.org/wiki/Developers/Maintainers#Key_extensions ?



I'll leave that for Yaron to do I guess. :-)

Is that list meant to contain all extensions that are used on more than 
400 wikis? Because that looks to be 230 at the moment, going on 
WikiApiary numbers [1]. It seems like a hard list to keep up to date, 
and it doesn't add any info that's not already available elsewhere.


I have a vague memory that there's some tool that reads 
[[Developers/Maintainers]] though, and (maybe?) adds reviewers on Gerrit 
based on it. Am I imagining that?


[1] 
https://wikiapiary.com/w/index.php?title=Special%3AAsk=%5B%5BCategory%3AExtension%5D%5D%0D%0A%5B%5BHas+website+count%3A%3A%3E400%5D%5D=%3FHas+website+count%0D%0A%3FHas+author%0D%0A=yes%5Bformat%5D=broadtable_num=_num=ASC%5Blimit%5D=500%5Boffset%5D=%5Blink%5D=all%5Bsort%5D=%5Bheaders%5D=show%5Bmainlabel%5D=%5Bintro%5D=%5Boutro%5D=%5Bsearchlabel%5D=...+further+results%5Bdefault%5D=%5Bclass%5D=sortable+wikitable+smwtable%5Bsep%5D==yes


___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

On Thu, 2019-06-20 at 15:47 +0800, Sam Wilson wrote:
> Thanks Andre, I like these emails.
> 
> Where does the 'Maintainers/Stewards' value come from? Because the
> two 
> Cargo ones should probably be allocated Yaron Koren, who's listed
> both 
> on the extension page on mediawiki.org and is the only member of the 
> extension-Cargo group in Gerrit.

Thanks for the feedback! Maybe Cargo (et al) should be added under
https://www.mediawiki.org/wiki/Developers/Maintainers#Key_extensions ?

Plus you finally made me find "People > List Groups" in the Gerrit UI.
Might come handy for future editions. :)

andre
-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Sam Wilson]

Thanks Andre, I like these emails.

Where does the 'Maintainers/Stewards' value come from? Because the two 
Cargo ones should probably be allocated Yaron Koren, who's listed both 
on the extension page on mediawiki.org and is the only member of the 
extension-Cargo group in Gerrit.


I only ask, because I'm never sure quite what role
https://www.mediawiki.org/wiki/Developers/Maintainers
has in determining these things.

On 6/20/19 3:08 PM, Andre Klapper wrote:

Summary of worst repos with more than one patch waiting:
5x WikiPEG. 4x Parsoid. 3x UnblockMe. 2x Cargo. 2x PageForms. 2x MW Core.


CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):


* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
** Added new feature: order virtual fields by the order they inserted to list 
on cargo store
** 2018-December-17
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/511394/
** Added 'bibtex' export format.
** 2019-May-24
** Maintainers/Stewards: ???

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/GoogleAnalyticsMetrics/+/485488/
** fix: non-ascii page title bug
** 2019-January-20
** Maintainers/Stewards: Tosfos (as per Extension page)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OdbcDatabase/+/487568/
** Fix OdbcDatabase to work under 1.32 and with current ExternalData
** 2019-February-02
** Maintainers/Stewards: ??? (Extension page says "Unmaintained")

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Newsletter/+/501019/
** Hide View Source Tab in Newsletter namespace
** 2019-April-30
** Maintainers/Stewards: ??? (Unassigned)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508045/
** Edit Project Config
* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/510791/
** Adding files
* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508047/
** Adding files
** Author is also extension author; author might need to get aware of
https://www.mediawiki.org/wiki/Gerrit/Privilege_policy#Requesting_Gerrit_privileges
or what's the policy on self-merging these days?

* https://gerrit.wikimedia.org/r/#/c/pywikibot/core/+/508092/
** remove dewiki archive template from edit restrictions
** 2019-May-05
** Maintainers/Stewards: ??? (not sure where Pywikibot lists that)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508165/
** Make autolink matching more efficient
** 2019-May-06
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508176/
** All NULL in flattenIfArray and remove superfluous return statements
** 2019-May-06
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512628/
** cache rule generator optimization & formatting
** 2019-May-27
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512632/
** cache rule generator clean-up
** 2019-May-27
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/507821/
** Modify "SiteExporter.php" and "SiteImporter.php" to handle "LanguageCode" 
property as well
** 2019-May-06
** Maintainers/Stewards: MediaWiki Platform Team

* https://gerrit.wikimedia.org/r/#/c/operations/mediawiki-config/+/508130/
** Create new http://www.mediawiki.org/xml/sitelist-1.1/ to reference 
sitelist-1.1.xsd
** 2019-May-07
** Maintainers/Stewards: MediaWiki Platform Team ?

* https://gerrit.wikimedia.org/r/#/c/operations/dns/+/508979/
** Revert "Convert most DYNA into 1H CNAME records"
** 2019-May-09
** Maintainers/Stewards: SRE

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/429533/
** ApiEditPage: Don't swap undo and undoafter parameters
** 2019-May-11
** Maintainers/Stewards: MediaWiki Platform Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/509629/
** Minimize multiple instance forms when using the Header Tabs extension
** 2019-May-13
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/512642/
** Fix autocomplete filter function for no \w chars.
** 2019-May-27
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510070/
** WikiPEG character class optimization
** 2019-May-14
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510311/
** doc update
** 2019-May-27
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/511820/
** Drop legacy & unused code (T224084). Avoid use of "discard" (T224085)
** 2019-May-22
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516455/
** chore: remove dead code and update .gitignore
** 2019-June-11
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516457/
** a-b benchmark tests
** 2019-June-11
** 

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

Summary of worst repos with more than one patch waiting:
5x WikiPEG. 4x Parsoid. 3x UnblockMe. 2x Cargo. 2x PageForms. 2x MW Core.


CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):


* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
** Added new feature: order virtual fields by the order they inserted to list 
on cargo store
** 2018-December-17
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/511394/
** Added 'bibtex' export format.
** 2019-May-24
** Maintainers/Stewards: ???

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/GoogleAnalyticsMetrics/+/485488/
** fix: non-ascii page title bug
** 2019-January-20
** Maintainers/Stewards: Tosfos (as per Extension page)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OdbcDatabase/+/487568/
** Fix OdbcDatabase to work under 1.32 and with current ExternalData
** 2019-February-02
** Maintainers/Stewards: ??? (Extension page says "Unmaintained")

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Newsletter/+/501019/
** Hide View Source Tab in Newsletter namespace
** 2019-April-30
** Maintainers/Stewards: ??? (Unassigned)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508045/
** Edit Project Config
* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/510791/
** Adding files
* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/UnblockMe/+/508047/
** Adding files
** Author is also extension author; author might need to get aware of
https://www.mediawiki.org/wiki/Gerrit/Privilege_policy#Requesting_Gerrit_privileges
 
or what's the policy on self-merging these days?

* https://gerrit.wikimedia.org/r/#/c/pywikibot/core/+/508092/
** remove dewiki archive template from edit restrictions
** 2019-May-05
** Maintainers/Stewards: ??? (not sure where Pywikibot lists that)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508165/
** Make autolink matching more efficient
** 2019-May-06
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/508176/
** All NULL in flattenIfArray and remove superfluous return statements
** 2019-May-06
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512628/
** cache rule generator optimization & formatting
** 2019-May-27
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/parsoid/+/512632/
** cache rule generator clean-up 
** 2019-May-27
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/507821/
** Modify "SiteExporter.php" and "SiteImporter.php" to handle "LanguageCode" 
property as well
** 2019-May-06
** Maintainers/Stewards: MediaWiki Platform Team

* https://gerrit.wikimedia.org/r/#/c/operations/mediawiki-config/+/508130/
** Create new http://www.mediawiki.org/xml/sitelist-1.1/ to reference 
sitelist-1.1.xsd
** 2019-May-07
** Maintainers/Stewards: MediaWiki Platform Team ?

* https://gerrit.wikimedia.org/r/#/c/operations/dns/+/508979/
** Revert "Convert most DYNA into 1H CNAME records"
** 2019-May-09
** Maintainers/Stewards: SRE

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/429533/
** ApiEditPage: Don't swap undo and undoafter parameters
** 2019-May-11
** Maintainers/Stewards: MediaWiki Platform Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/509629/
** Minimize multiple instance forms when using the Header Tabs extension
** 2019-May-13
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/512642/
** Fix autocomplete filter function for no \w chars.
** 2019-May-27
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510070/
** WikiPEG character class optimization
** 2019-May-14
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/510311/
** doc update
** 2019-May-27
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/511820/
** Drop legacy & unused code (T224084). Avoid use of "discard" (T224085)
** 2019-May-22
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516455/
** chore: remove dead code and update .gitignore
** 2019-June-11
** Maintainers/Stewards: Parsing Team ?

* https://gerrit.wikimedia.org/r/#/c/wikipeg/+/516457/
** a-b benchmark tests
** 2019-June-11
** Maintainers/Stewards: Parsing Team ?

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ParserFunctions/+/511030/
** Introduce an additional parameter to #time and #timel functions to allow the 
wrapping of the result into  HTML tag
** 2019-May-20
** Maintainers/Stewards: Parsing Team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/VEForAll/+/513913/
** fix update all editors before save
** 2019-June-02
** Maintainers/Stewards: ???



CR+1: Please help 

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the code, and 
decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
** Added new feature: order virtual fields by the order they inserted to list 
on cargo store
** 2018-December-17
** Maintainers/Stewards: ???

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/GoogleAnalyticsMetrics/+/485488/
** fix: non-ascii page title bug
** 2019-January-20
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OdbcDatabase/+/487568/
** Fix OdbcDatabase to work under 1.32 and with current ExternalData
** 2019-February-02
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FirstSteps/+/492460/
** Convert FirstSteps to use extension registration
** 2019-February-27
** Maintainers/Stewards: ??? (unmaintained?)

* https://gerrit.wikimedia.org/r/#/c/pywikibot/core/+/492485/
** [bugfix] Remove link to /edithistory from page's edithistory summary.
** 2019-March-17
** Maintainers/Stewards: ??? (not sure where Pywikibot lists that)

* https://gerrit.wikimedia.org/r/#/c/operations/puppet/+/498773/
** db::views: Bring back abuse_filter_history table
** 2019-March-25
** Maintainers/Stewards: WMCS?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Newsletter/+/499292/
** Split API messages to a separate file 
** 2019-March-26
** Maintainers/Stewards: ???



CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PipeEscape/+/486043/
** Convert PipeEscape to use extension registration
** 2019-April-20
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Push/+/492443/
** Use addJsConfigVars instead of addInlineScript
** 2019-April-09
** Maintainers/Stewards: ??? (mw.org says Unmaintained)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/429533/
** ApiEditPage: Don't swap undo and undoafter parameters
** 2019-April-01
** Maintainers/Stewards: MediaWiki Platform team?



Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!



Of last time's 20 listed patches, 
1 got -1ed by Santhosh;
3 got +1ed by D3rick01, Thiemo Kreuz, WMDE-Fisch, Pastakhov;
9 got +2ed by D3rick01, Jforrester, Brian Wolff, Xqt, Volans, MarcoAurelio, 
EBernhardson, Umherirrender;
1 got abandoned.



Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Jaime Crespo]

For the couple I am slightly familiar with (apologies if I am mistaken):

On Thu, Apr 4, 2019 at 11:11 AM Andre Klapper  wrote:
> * https://gerrit.wikimedia.org/r/#/c/operations/software/cumin/+/497312/
> ** allow running cumin as a regular user
> ** 2019-March-18
> ** Maintainers/Stewards: WMF SRE?

Cumin is maintained by the software automation subteam of SRE.

> * https://gerrit.wikimedia.org/r/#/c/operations/puppet/+/498773/
> ** db::views: Bring back abuse_filter_history table
> ** 2019-March-25
> ** Maintainers/Stewards: WMF SRE?

Wikireplica views are maintained by the WMCS team (not SRE), although
they may or may not need assistance by SREs/security, depending on the
proposed changes.

I will bring these 2 to their attention.

Thanks for the heads up, Andre

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the code, and 
decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/Material/+/471766/
** Add scrollbar:: to view more content on sidebar
** 2018-November-06
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472184/
** Avoid hitting DB in Category getters when they have the required info
** 2018-November-23 (rebased on 2019-March-10)
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
** Added new feature: order virtual fields by the order they inserted to list 
on cargo store
** 2018-December-17
** Maintainers/Stewards: ???

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/GoogleAnalyticsMetrics/+/485488/
** fix: non-ascii page title bug
** 2019-January-20
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PipeEscape/+/486043/
** Convert PipeEscape to use extension registration
** 2019-January-23
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OdbcDatabase/+/487568/
** Fix OdbcDatabase to work under 1.32 and with current ExternalData
** 2019-February-02
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Push/+/492443/
** Use addJsConfigVars instead of addInlineScript
** 2019-February-23
** Maintainers/Stewards: ??? (mw.org says Unmaintained)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/SearchStats/+/492455/
** Add i18n key searchstats
** 2019-February-23
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FirstSteps/+/492460/
** Convert FirstSteps to use extension registration
** 2019-February-27
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OATHAuth/+/492436/
** Added missing disableoathforuser message
** 2019-March-03
** Maintainers/Stewards: ??? ("Unassigned")

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/495439/
** Add PHPDocs to CargoViewData functions
** 2019-March-09
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MultiMaps/+/494860/
** Extension MultiMaps: Adding support for multiple URL in Leaflet
** 2019-March-07
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/pywikibot/core/+/492485/
** [bugfix] Remove link to /edithistory from page's edithistory summary.
** 2019-March-17
** Maintainers/Stewards: ??? (not sure where Pywikibot lists that)

* https://gerrit.wikimedia.org/r/#/c/operations/software/cumin/+/497312/
** allow running cumin as a regular user
** 2019-March-18
** Maintainers/Stewards: WMF SRE?

* https://gerrit.wikimedia.org/r/#/c/operations/puppet/+/498773/
** db::views: Bring back abuse_filter_history table
** 2019-March-25
** Maintainers/Stewards: WMF SRE?

* https://gerrit.wikimedia.org/r/#/c/pywikibot/core/+/497646/
** pywikibot: Add support for property creation
** 2019-March-25 (but with a "WIP" flag)
** Maintainers/Stewards: ??? (not sure where Pywikibot lists that)


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/WhiteSpace/+/493168/
** Convert WhiteSpace to use extension registration
** 2019-February-27
** Maintainers/Stewards: ???

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/GlobalContributions/+/492444/
** Use plaintextParams directly instead of using rawParams()
** 2019-February-27 (but blocked on non-removed "WIP" flag)
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/CloseWikis/+/492449/
** SpecialCloseWiki: Replace the usage of global variables
** 2019-March-21
** Maintainers/Stewards: ???

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/LastUserLogin/+/492441/
** LastUserLogin: Replace the usage of global variables
** 2019-March-21
** Maintainers/Stewards: ???


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 11 listed patches, 
2 got -1ed by anomie and Tgr;
2 got +2ed by by Daniel Kinzler and MatmaRex;
1 got rebased and is back to CR0.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/462155/
** Adds alternative file icon paths
** 2018-October-14
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/Material/+/471766/
** Add scrollbar:: to view more content on sidebar
** 2018-November-06
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
** Added new feature: order virtual fields by the order they inserted to list 
on cargo store
** 2018-December-17
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/SecurePoll/+/463610/
** Change SecurePoll auth-api.php to a MediaWiki Action API module
** 2019-Jaunary-12
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/473769/
** Fully utilize LinkTarget passed to getRevisionByTitle
** 2019-January-08
** Maintainers/Stewards: MediaWiki Platform team?

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/GoogleAnalyticsMetrics/+/485488/
** fix: non-ascii page title bug
** 2019-January-20
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OdbcDatabase/+/487568/
** Fix OdbcDatabase to work under 1.32 and with current ExternalData
** 2019-February-02
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PipeEscape/+/486043/
** Convert PipeEscape to use extension registration
** 2019-January-23
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/FirstSteps/+/492460/
** Convert FirstSteps to use extension registration
** 2019-February-27
** Maintainers/Stewards: ???


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472758/
** Added Id to the input box
** 2019-January-07
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472184/
** Avoid hitting DB in Category getters when they have the required info
** 2018-November-23
** Maintainers/Stewards: MediaWiki Platform team?


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 12 listed patches, 
4 got abandoned by HerrTaschenbier, Zfilipin, Eileen; 
1 got -2ed by EBernhardson;
2 got 1-ed by Thiemo, cscott.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/462155/
** Adds alternative file icon paths
** 2018-October-14
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/avro-php/+/466838/
** Convert Avro-PHP to PSR-Compliant Project
** 2018-October-18
** Maintainers/Stewards: Analytics / Discovery (?)

* https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/Material/+/471766/
** Add scrollbar:: to view more content on sidebar
** 2018-November-06
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472758/
** Added Id to the input box
** 2018-November-10
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/474769/
** Add Special:CategoriesToEmpty
** 2018-November-20
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472184/
** Avoid hitting DB in Category getters when they have the required info
** 2018-November-23
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/operations/mediawiki-config/+/478498/
** Upload HD logos for several projects
** 2018-December-12
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/wikimedia/fundraising/crm/+/479533/
** Update ctools to 1.1.4
** 2018-December-13
** Maintainers/Stewards: Fundraising Tech

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cargo/+/480039/
** Added new feature: order virtual fields by the order they inserted to list 
on cargo store
** 2018-December-17
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Echo/+/471910/
** Test: notify a new user with welcome message
** 2018-December-17
** Maintainers/Stewards: Growth team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/481313/
** Allow TOC to "duplicate" existing anchors in MediaWiki
** 2018-December-25
** Maintainers/Stewards: MediaWiki Platform team?


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PluggableAuth/+/468722/
** Adding separate option for local API login
** 2018-October-20
** Maintainers/Stewards: ???


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 10 listed patches, 3 got abandoned by Zfilipin,
Jforrester, Sbassett.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/462155/
** Adds alternative file icon paths
** 2018-October-14
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/avro-php/+/466838/
** Convert Avro-PHP to PSR-Compliant Project
** 2018-Ocotber-18
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Echo/+/471909/
** Test: notify a new user with welcome message
** 2018-November-06
** Maintainers/Stewards: Growth team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/Material/+/471766/
** Add scrollbar:: to view more content on sidebar
** 2018-November-06
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472758/
** Added Id to the input box
** 2018-November-10
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472491/
** MediaWiki Special Pages CSS : Add indentation to content under header
** 2018-November-08
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/474769/
** Add Special:CategoriesToEmpty
** 2018-November-20
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472184/
** Avoid hitting DB in Category getters when they have the required info
** 2018-November-23
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/475798/
** Revert Special:PasswordReset: Make the user field a user lookahead field
** 2018-November-26
** Maintainers/Stewards: MediaWiki Platform team?


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PluggableAuth/+/468722/
** Adding separate option for local API login
** 2018-October-20
** Maintainers/Stewards: ???


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 15 listed patches, 1 got merged by Yaron Koren, 5
received reviews by Jack Phoenix, Daimona Eaytoy, Legoktm, Anomie, and
VolkerE, 3 got abandoned.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):


* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201177/
** Add average AbuseFilter run time and conditions to display table
** 2018-October-16
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201171/
** Expand stat tracking in AbuseFilter
** 2018-October-16
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/462155/
** Adds alternative file icon paths
** 2018-October-14
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201104/
** Overhaul AbuseFilter internal profiling system
** 2018-October-24
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/avro-php/+/466838/
** Convert Avro-PHP to PSR-Compliant Project
** 2018-Ocotber-18
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/469503/
** MWExceptionRenderer: Improve the message shown when a database error occurs
** 2018-October-27
** Maintainers/Stewards: MediaWiki Platform team?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Echo/+/470729/
** Update Echo.page.js::check for welcome element, Fix Echo.js::added page 
load, Add notification_welcome test
** 2018-November-06
** Maintainers/Stewards: Growth team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Echo/+/471909/
** Test: notify a new user with welcome message
** 2018-November-06
** Maintainers/Stewards: Growth team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Echo/+/471910/
** Test: notify a new user with welcome message
** 2018-November-06
** Maintainers/Stewards: Growth team

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/RevisionSlider/+/471538/
** Change color to darker shade
** 2018-November-04
** Maintainers/Stewards: WMDE Engineering

* https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/Material/+/471766/
** Add scrollbar:: to view more content on sidebar
** 2018-November-06
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/472491/
** MediaWiki Special Pages CSS : Add indentation to content under header
** 2018-November-08
** Maintainers/Stewards: MediaWiki Platform team?


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/VEForAll/+/454560/
** Fix composer package name to match conventions
** 2018-September-07
** Maintainers/Stewards: ???

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PluggableAuth/+/468722/
** Adding separate option for local API login
** 2018-October-20
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/Refreshed/+/470621/
** Fix focus on 'More' drop down when tabbing in skin Refreshed.
** 2018-October-31
** Maintainers/Stewards: ???


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 6 listed patches, 2 received reviews by Addshore and
Nikerabbit.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):


* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201177/
** Add average AbuseFilter run time and conditions to display table
** 2018-October-16
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201171/
** Expand stat tracking in AbuseFilter
** 2018-October-16
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/462155/
** Adds 'GetFileIconThumb' hook to file page
** 2018-October-14
** Maintainers/Stewards: Reading > Multimedia team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Translate/+/463335/
** New feature: Google Translate translation provider
** 2018-October-01
** Maintainers/Stewards: Language engineering


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/453821/
** Add monolingual language code hbo
** 2018-August-27
** Maintainers/Stewards: WMDE Engineering

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/VEForAll/+/454560/
** Fix composer package name to match conventions
** 2018-September-07
** Maintainers/Stewards: ???


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 6 listed patches, 1 received reviews by Jamesmontalvo3,
Cicalse, Anomie.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper  |  ak...@gmx.net
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):


* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201104/
** Overhaul AbuseFilter internal profiling system
** 2018-September-09
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201177/
** Add average AbuseFilter run time and conditions to display table
** 2018-September-09
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201171/
** Expand stat tracking in AbuseFilter
** 2018-September-09
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/462155/
** Adds 'GetFileIconThumb' hook to file page
** 2018-September-25
** Maintainers/Stewards: Reading > Multimedia team


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/453821/
** Add monolingual language code hbo
** 2018-August-27
** Maintainers/Stewards: WMDE Engineering

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/VEForAll/+/454560/
** Fix composer package name to match conventions
** 2018-September-07
** Maintainers/Stewards: ???


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 11 listed patches, 2 got merged thanks to Mukunda and
JForrester, 3 changed to CR-1 thanks to bawolff, TheDJ, Eranroz,
JJMC89.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper  |  ak...@gmx.net
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MyVariables/+/454427/
** Update to modern extension loading and reorganize and apply coding standards
** 2018-August-28
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/CentralAuth/+/457498/
** Display global rename log in "Previous global account changes"
** 2018-September-06
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OATHAuth/+/452020/
** Temporarily disable certain rights if 2FA isn't enabled for the user
** 2018-September-08
** Maintainers/Stewards: ???

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cite/+/459371/
** rm unnecessary overwrites
** 2018-September-09
** Maintainers/Stewards: Contributors > Editing team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Cite/+/459372/
** add dpub-aria endnote roles
** 2018-September-09
** Maintainers/Stewards: Contributors > Editing team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201104/
** Overhaul AbuseFilter internal profiling system
** 2018-September-09
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201177/
** Add average AbuseFilter run time and conditions to display table
** 2018-September-09
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/AbuseFilter/+/201171/
** Expand stat tracking in AbuseFilter
** 2018-September-09
** Maintainers/Stewards: MediaWiki Platform team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/458909/
** Add a way to exclude categories from Special:UnusedCategories
** 2018-September-10
** Maintainers/Stewards: ???


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/453821/
** Add monolingual language code hbo
** 2018-August-27
** Maintainers/Stewards: WMDE Engineering

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/VEForAll/+/454560/
** Fix composer package name to match conventions
** 2018-September-07
** Maintainers/Stewards: ???


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last week's 3 listed patches, 1 changed to CR-1 thanks to Jforrester
and Matmarex.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper  |  ak...@gmx.net
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MyVariables/+/454427/
** Update to modern extension loading and reorganize and apply coding standards
** 2018-August-28
** Maintainers/Stewards: ?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/TemplateData/+/457072/
** Remove empty Default, Example and Auto value sections of the output table
** 2018-September-02
** Maintainers/Stewards: Contributors > Editing team


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/453821/
** Add monolingual language code hbo
** 2018-August-27
** Maintainers/Stewards: WMDE Engineering


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 4 listed patch, 1 changed to CR-1 thanks to JForrester,
1 changed to CR+1 thanks to Foxtrott.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper  |  ak...@gmx.net
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* 
https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/SpamBlacklist/+/423612/
** Revert "Revert "add extension name in composer.json""
** 2018-July-04
** Maintainers/Stewards: Contributors > Editing team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/VEForAll/+/454560/
** fix composer package name to match conventions
** 2018-August-22
** Maintainers/Stewards: ?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MyVariables/+/454427/
** Update to modern extension loading and reorganize and apply coding standards
** 2018-August-28
** Maintainers/Stewards: ?

CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Wikibase/+/453821/
** Add monolingual language code hbo
** 2018-August-27
** Maintainers/Stewards: WMDE Engineering


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last time's 1 listed patch, 1 changed to Verified-1.


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D0+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/164049/
** Add Tatar LanguageConverter
** 2018-May-24 (though commented by Thiemo on 2018-Jul-05)
** Maintainers/Stewards: Contributors > Parsing team


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last edition's 3 listed patches, 2 got merged.
Thanks to James Forrester and Yaron Koren!


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253D1+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0
-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/164049/
** Add Tatar LanguageConverter
** 2018-May-24 (though commented by Thiemo on 2018-Jul-05)
** Maintainers/Stewards: Contributors > Parsing team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Drafts/+/443128/
** Adding code standards configuration for phpcs to drafts.
** 2018-Jul-09
** Maintainers/Stewards: ?


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ApprovedRevs/+/434052/
** Add support for custom content types other than WikitextContent
** 2018-Jun-17
** Maintainers/Stewards: ?


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last edition's 4 listed patches, 2 got merged.
Thanks to Fisch and Mholloway!


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253D1+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0
-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/



___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Stephane Bisson]

>
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/428129/
> ** Fix the bug for dates between 1912 and 1941 in Thai language
> ** 2018-Jun-13
> ** Maintainers/Stewards: Contributors > Global Collaboration team ?
>

#REDIRECT Language team

The Global Collab team was unmerged into Collab and Language, and Collab
was transformed into Growth ;)

-- 
Stephane Bisson
Wikimedia Foundation
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Bernd Sitzmann]

If it's blocked this should get a -1 with the reason why.

On Thu, Jul 5, 2018 at 8:03 AM Michael Holloway 
wrote:

> On Thu, Jul 5, 2018 at 9:46 AM Andre Klapper 
> wrote:
>
> > *
> >
> https://gerrit.wikimedia.org/r/#/c/mediawiki/services/mobileapps/+/441517/
> > ** Metadata: Adds per-variant display_title
> > ** 2018-Jun-25
> > ** Maintainers/Stewards: Reading Infrastructure / Services ?
> >
>
> This patch is on RI's radar but blocked on a product decision.
>
> That said, thanks for sending out these notices!
>
> -mdh
> ___
> Wikitech-l mailing list
> Wikitech-l@lists.wikimedia.org
> https://lists.wikimedia.org/mailman/listinfo/wikitech-l
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Michael Holloway]

On Thu, Jul 5, 2018 at 9:46 AM Andre Klapper  wrote:

> *
> https://gerrit.wikimedia.org/r/#/c/mediawiki/services/mobileapps/+/441517/
> ** Metadata: Adds per-variant display_title
> ** 2018-Jun-25
> ** Maintainers/Stewards: Reading Infrastructure / Services ?
>

This patch is on RI's radar but blocked on a product decision.

That said, thanks for sending out these notices!

-mdh
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/164049/
** Add Tatar LanguageConverter
** 2018-May-24
** Maintainers/Stewards: Contributors > Parsing team


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ApprovedRevs/+/434052/
** Add support for custom content types other than WikitextContent
** 2018-Jun-17
** Maintainers/Stewards: ?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/428129/
** Fix the bug for dates between 1912 and 1941 in Thai language
** 2018-Jun-13
** Maintainers/Stewards: Contributors > Global Collaboration team ?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/services/mobileapps/+/441517/
** Metadata: Adds per-variant display_title
** 2018-Jun-25
** Maintainers/Stewards: Reading Infrastructure / Services ?


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Of last week's 5 listed patches, 3 got a CR-1 and one got a CR+1.
Thanks to Thiemo!


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253D1+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/


___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Paladox]

 I wonder if we could create a polygerrit plugin and just display a daily or 
weekly list of changes for users to review.
On Friday, 29 June 2018, 01:36:50 BST, Jean-Rene Branaa 
 wrote:  
 
 Hola,

I like this, thanks for doing it.  As part of the Stewardship push,
I've yet to start measuring/baselining responsiveness, this will help
bring attention to reviews that are languishing.  I also like the fact
that you're specifying the stewards (or lack there of). Again, helpful
for me too :-)

Cheers,

JR

On Thu, Jun 28, 2018 at 2:59 AM Andre Klapper  wrote:
>
> (Resurrecting the weekly posts from late 2016. Feedback welcome.)
>
>
> CR0: Please review and provide guidance if you are familiar with the
> code, and decide (CR±1 or CR±2):
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/431320/
> ** Trim whitespaces from the end of the text
> ** 2018-May-06
> ** Maintainers/Stewards: Contributors > Parsing team
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/164049/
> ** Add Tatar LanguageConverter
> ** 2018-May-24
> ** Maintainers/Stewards: Contributors > Parsing team
>
> * 
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ApprovedRevs/+/434052/
> ** Add support for custom content types other than WikitextContent
> ** 2018-Jun-17
> ** Maintainers/Stewards: ?
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/441021/
> ** edit-with-form-tab-not-displaying
> ** 2018-Jun-19
> ** Maintainers/Stewards: ?
>
>
> CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:
>
> * 
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ExternalData/+/421525/
> ** Cast the variable to an array to prevent PHP warnings
> ** 2018-March-23
> ** Maintainers/Stewards: ?
>
>
> Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
> you can get notified of new patches in your code areas of interest.
>
> Thanks in advance for your reviews!
>
>
> Maintainers/Stewards data taken from 
> https://www.mediawiki.org/wiki/Developers/Maintainers
> CR0 source: 
> https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253D1+label:Code-Review%253D0
> CR+1 source: 
> https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0
>
> --
> Andre Klapper | Bugwrangler / Developer Advocate
> https://blogs.gnome.org/aklapper/
>
>
> ___
> Wikitech-l mailing list
> Wikitech-l@lists.wikimedia.org
> https://lists.wikimedia.org/mailman/listinfo/wikitech-l

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l  
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Jean-Rene Branaa]

Hola,

I like this, thanks for doing it.  As part of the Stewardship push,
I've yet to start measuring/baselining responsiveness, this will help
bring attention to reviews that are languishing.  I also like the fact
that you're specifying the stewards (or lack there of). Again, helpful
for me too :-)

Cheers,

JR

On Thu, Jun 28, 2018 at 2:59 AM Andre Klapper  wrote:
>
> (Resurrecting the weekly posts from late 2016. Feedback welcome.)
>
>
> CR0: Please review and provide guidance if you are familiar with the
> code, and decide (CR±1 or CR±2):
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/431320/
> ** Trim whitespaces from the end of the text
> ** 2018-May-06
> ** Maintainers/Stewards: Contributors > Parsing team
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/164049/
> ** Add Tatar LanguageConverter
> ** 2018-May-24
> ** Maintainers/Stewards: Contributors > Parsing team
>
> * 
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ApprovedRevs/+/434052/
> ** Add support for custom content types other than WikitextContent
> ** 2018-Jun-17
> ** Maintainers/Stewards: ?
>
> * https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/441021/
> ** edit-with-form-tab-not-displaying
> ** 2018-Jun-19
> ** Maintainers/Stewards: ?
>
>
> CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:
>
> * 
> https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ExternalData/+/421525/
> ** Cast the variable to an array to prevent PHP warnings
> ** 2018-March-23
> ** Maintainers/Stewards: ?
>
>
> Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
> you can get notified of new patches in your code areas of interest.
>
> Thanks in advance for your reviews!
>
>
> Maintainers/Stewards data taken from 
> https://www.mediawiki.org/wiki/Developers/Maintainers
> CR0 source: 
> https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253D1+label:Code-Review%253D0
> CR+1 source: 
> https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0
>
> --
> Andre Klapper | Bugwrangler / Developer Advocate
> https://blogs.gnome.org/aklapper/
>
>
> ___
> Wikitech-l mailing list
> Wikitech-l@lists.wikimedia.org
> https://lists.wikimedia.org/mailman/listinfo/wikitech-l

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

[Wikitech-l] Patchsets by new Gerrit contributors waiting for code review and/or merge

[Andre Klapper]

(Resurrecting the weekly posts from late 2016. Feedback welcome.)


CR0: Please review and provide guidance if you are familiar with the
code, and decide (CR±1 or CR±2):

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/431320/
** Trim whitespaces from the end of the text
** 2018-May-06
** Maintainers/Stewards: Contributors > Parsing team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/core/+/164049/
** Add Tatar LanguageConverter
** 2018-May-24
** Maintainers/Stewards: Contributors > Parsing team

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ApprovedRevs/+/434052/
** Add support for custom content types other than WikitextContent
** 2018-Jun-17
** Maintainers/Stewards: ?

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/PageForms/+/441021/
** edit-with-form-tab-not-displaying
** 2018-Jun-19
** Maintainers/Stewards: ?


CR+1: Please help make a decision (CR±1, CR±2) on these CR+1 patches:

* https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/ExternalData/+/421525/
** Cast the variable to an array to prevent PHP warnings
** 2018-March-23
** Maintainers/Stewards: ?


Read https://www.mediawiki.org/wiki/Gerrit/Code_review#By_project how
you can get notified of new patches in your code areas of interest.

Thanks in advance for your reviews!


Maintainers/Stewards data taken from 
https://www.mediawiki.org/wiki/Developers/Maintainers
CR0 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253D1+label:Code-Review%253D0
CR+1 source: 
https://gerrit.wikimedia.org/r/#/q/ownerin:newcomers+status:open+label:Verified%253E%253D1+label:Code-Review%253E%253D%252B1+-label:Code-Review%253C%253D0

-- 
Andre Klapper | Bugwrangler / Developer Advocate
https://blogs.gnome.org/aklapper/


___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l