Patchsets by new Gerrit contributors waiting for code review and/or merge

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Patchsets by new Gerrit contributors waiting for code review and/or merge

Andre Klapper-2
(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
[hidden email]
https://lists.wikimedia.org/mailman/listinfo/wikitech-l
Reply | Threaded
Open this post in threaded view
|

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

_______________________________________________
Wikitech-l mailing list
[hidden email]
https://lists.wikimedia.org/mailman/listinfo/wikitech-l
Reply | Threaded
Open this post in threaded view
|

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

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