User:AKlapper (WMF)/Code Review/Weekly Email

From mediawiki.org

Realistically speaking: Monthly, not weekly.

Email template[edit]

Notes:

  • For populating the two lists of patches, consider using age:6d as an additional Gerrit query parameter as we do not want to list items that did receive updates from reviewers within the last week. However, an update could have also been a ping comment from the patch author, so better manually check the last comments of each patch to decide whether to list them or not.
  • Look up "Maintainers/Stewards" from mw:Developers/Maintainers, simply set "?" if not listed
  • Email to be sent to wikitech-l@ on Thursdays.
Patchsets by new Gerrit contributors waiting for code review and/or merge

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

* URL
** Topic
** last updated/reviewed
** Maintainers/Stewards:


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

* URL
** Topic
** last updated/reviewed
** 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 X listed patches, X got merged.
Thanks to Alice and Bob!


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
Retrieved from "https://www.mediawiki.org/w/index.php?title=User:AKlapper_(WMF)/Code_Review/Weekly_Email&oldid=3355705"