Git/Reviewers

Manual on MediaWiki Tools

Gerrit Reviewer Bot Release status: stable
Description	Adds reviewers to Gerrit patchsets Log of recent actions
Author(s)	User:valhallasw Please send me a talk page message or e-mail if the bot is malfunctioning.
License	MIT
Download	https://github.com/valhallasw/gerrit-reviewer-bot

 Note: If you are looking for a list of people that could review your patch, please see Developers/Maintainers.

This is a page where people can register to be automatically added as reviewer to newly opened changes in Gerrit or Gitlab depending on specific conditions. After registering here, reviewer-bot will add you as reviewer to every change that matches the project and file filter. For Gitlab, you will get an "@" mention in the merge request.

Gerrit supports a similar feature by itself (including filtering by modified file name and words appearing in commit message), see Gerrit/watched projects and https://gerrit.wikimedia.org/r/settings/projects. Likewise, GitLab supports enabling notifications on projects, groups, individual merge requests, etc. See https://docs.gitlab.com/ee/user/profile/notifications.html. GitLab does not support filename-based matching, so this page may still be useful to you.

Each project has a header with the Gerrit or Gitlab project name, followed by a list of people (using Template:Gerrit-reviewer or Template:Gitlab-mention), for example:

=== test/mediawiki/extensions/examples ===
* {{Gerrit-reviewer|User 2}}
* {{Gerrit-reviewer|User 1|file_regexp=<nowiki>i18n</nowiki>}}

The file_regexp parameter can be used to filter for changes in specific files:

* {{Gerrit-reviewer|User 1|file_regexp=<nowiki>i18n</nowiki>}}

will watch for changes where a file with 'i18n' in the name is changed, while

* {{Gerrit-reviewer|User 1|file_regexp=<nowiki>^includes/specials</nowiki>}}

will watch for changes in the includes/specials directory. You can test the regexp using [1] with IGNORECASE and DOTALL enabled. The <nowiki> tags are suggested but only required when the regexp contains characters that interfere with template parsing (e.g. | and }}).

An optional match_all_files parameter makes the bot only mark changes where the file_regexp matches all the changed files:

* {{Gerrit-reviewer|User 1|file_regexp=<nowiki>\.css|\.txt</nowiki>|match_all_files}}

This is useful since review scores apply to the whole changeset, rather than parts of it, so users that are comfortable with reviewing only a subsection of the changes wouldn't be able to review the whole patchset. For instance, front-end changes often touch both CSS and JavaScript files. A user who is confident about his CSS knowledge but not so much about his JavaScript ones can use match_all_files to restrict reviews attributed to them to those that only touch CSS files.

Projects are matched based on unix filename matching as implemented in the fnmatch module. The idea is to set a file_regexp, which filters changesets by changed files instead of repository.

• Zoranzoki21 (only commits where at least one of the changed files match /(bs|hr|sr\-e[cl])\.json/)
• Thcipriani (only commits where at least one of the changed files match /scap/scap\.cfg/)
• Revi (only commits where at least one of the changed files match /ko\.json/)
• Dr0ptp4kt (only commits where at least one of the changed files match /abstractSchemaChanges/)

• Majavah
• dcaro

Watch anything but integration/visualdiff which is maintained by another team.

• Hashar

• jbond (only commits where at least one of the changed files match /^dockerfiles/operations-puppet/)

• Kosta Harlan

• dcausse

• bearloga

• mglaser
• Robert Vogel
• ItSpiderman
• Zoranzoki21

• mglaser
• Robert Vogel
• ItSpiderman
• Zoranzoki21

• Daimona Eaytoy

• Urbanecm

• Sebastian Berlin (WMSE)
• Lokal Profil

• dcaro

• Ottomata

• Legoktm
• Volans

• Alex Paskulin

• Aaron Schulz (only commits where at least one of the changed files match /^(includes/(filebackend|db|objectcache|externalstore)/|includes/libs/|maintenance/archive/[a-zA-Z]+)/)
• Xcollazo (only commits where at least one of the changed files match /^includes/export/|^maintenance/includes/(Backup|TextPass|SevenZip)|^maintenance/(fetchText|dump)/)
• jebe (only commits where at least one of the changed files match /^includes/export/|^maintenance/includes/(Backup|TextPass|SevenZip)|^maintenance/(fetchText|dump)/)
• milimetric (only commits where at least one of the changed files match /^includes/export/|^maintenance/includes/(Backup|TextPass|SevenZip)|^maintenance/(fetchText|dump)/)
• Catrope (only commits where at least one of the changed files match /resources/src/vue//)
• cscott (only commits where at least one of the changed files match /[Pp]arser|virtualrest/)
• Cwhite (only commits where at least one of the changed files match /^((tests/phpunit/unit/)?includes/libs/Metrics)/)
• daniel (only commits where at least one of the changed files match /^includes/content//)
• dcausse (only commits where at least one of the changed files match /^includes/search//)
• Gergő Tisza (only commits where at least one of the changed files match /^includes/auth/|includes/session//)}}
• Huji (only commits where at least one of the changed files match /(Equivset|MessagesFa)/)
• Jdlrobson (only commits where at least one of the changed files match /^(includes/skins|includes/ResourceLoader/SkinModule\.php|resources/.*)/)
• Jjanes (only commits where at least one of the changed files match /\.sql$/)
• Kosta Harlan (only commits where at least one of the changed files match /^tests/phpunit/MediaWikiUnitTestCase.php/)
• Legoktm (only commits where at least one of the changed files match /registration|[Cc]onfig/)
• Liangent (only commits where at least one of the changed files match /^includes/language/(LanguageConverter\.php|converters/ZhConverter)|^includes/languages/(LanguageZh|data/ZhConversion\.php)|^maintenance/language/zhtable//)
• Revi (only commits where at least one of the changed files match /languages/messages/MessagesKo.php/)
• Samwilson (only commits where at least one of the changed files match /resources/src/mediawiki\.editRecovery/)
• Skizzerz (only commits where at least one of the changed files match /[Mm]ssql/)
• Tchanders (only commits where at least one of the changed files match /^includes/.*block/)
• Tpt (only commits where at least one of the changed files match /(DjVu|EditPage\.php)/)
• Urbanecm (only commits where at least one of the changed files match /(Messages(Cs|Sk)|includes/block/(UnblockUser|BlockUser|UserBlockCommandFactory|BlockPermissionChecker)|includes/specials/(SpecialBlock|SpecialUnblock).php)/)
• Winston Sung (only commits where at least one of the changed files match /LanguageCode|/Names\.php|LanguageConverter|/converters/|(Language|Messages)Zh|ZhConversion|^maintenance/language/zhtable//)
• Zabe (only commits where at least one of the changed files match /^includes/.*block/)
• Zppix (only commits where at least one of the changed files match /languages/l18n/en.json/)

• Ottomata

• Daimona Eaytoy
• Samtar

• Xcollazo
• jebe
• milimetric

• WMDE-Fisch

• mglaser
• Robert Vogel
• ItSpiderman

• Zoranzoki21

• Scardenasmolinar
• Jsn.sherman
• kgraessle

• Lewis Cawte

• Daimona Eaytoy
• cmelo
• MHORSEY
• Arinaigum (only commits where at least one of the changed files match /^db_patches/)

• DCausse
• EBernhardson
• TJones

• Huji

• Zabe
• Gergő Tisza

• ejegg
• mepps
• jgleeson
• XenoRyet
• cstone

• Wugapodes

• Huji
• Tchanders (only commits where at least one of the changed files match /^src/)
• Dreamy Jazz
• Kosta Harlan

• Winston Sung (only commits where at least one of the changed files match /LocalNames/)

• TheDJ
• Samtar

• MusikAnimal
• Samtar

• mglaser
• Robert Vogel
• ItSpiderman

• cscott

• cscott

• cscott

• cscott

• Cicalese

• Sergio Gimeno
• Urbanecm
• CSimiyu
• Michael Große

• samwilson
• MusikAnimal

• Ammarpad

• mglaser
• Robert Vogel
• ItSpiderman

• mglaser
• Robert Vogel
• ItSpiderman

• mglaser
• Robert Vogel
• ItSpiderman

• mglaser
• Robert Vogel
• ItSpiderman

• Cicalese

• MusikAnimal

• Cicalese

• ejegg
• XenoRyet
• jgleeson
• cstone
• Wfan

• mglaser
• Robert Vogel
• ItSpiderman

• WMDE-Fisch

• Cicalese

• Ottomata

• WMDE-Fisch

• WMDE-Fisch

• mglaser
• Robert Vogel
• ItSpiderman

• Ammarpad

• Samwilson

• TheDJ

• DannyS712

• Sergio Gimeno
• Urbanecm
• CSimiyu
• Michael Große

• phuedx

• Cicalese

• Kosta Harlan

• Kosta Harlan

• mglaser
• Robert Vogel
• ItSpiderman

• Kosta Harlan
• Dreamy Jazz

• Cicalese

• YOUR1

• TheDJ

• Robert Vogel
• Cicalese

• Robert Vogel
• Cicalese

• Robert Vogel

• Robert Vogel

• Robert Vogel

• Foxtrott

• Huji
• Samtar

• Physikerwelt
• AndreG-P
• Stegmujo

• Hcohl
• Physikerwelt
• AndreG-P

• wctaiwan

• Ostrzyciel

• Seb35

• Cparle

• Samwilson

• dbrant

• D3r1ck01 (only commits where at least one of the changed files match /^tests/phpunit/)

• mglaser
• Robert Vogel
• ItSpiderman

• Scardenasmolinar
• Jsn.sherman
• kgraessle

• Robert Vogel

• Gergő Tisza

• Gergő Tisza

• Lokal Profil

• Cicalese

• mglaser
• Robert Vogel
• ItSpiderman

• NovemLinguae
• MPGuy2824
• Samtar
• Jsn.sherman
• Scardenasmolinar
• Sohom Datta

• Cicalese

• Samtar
• MusikAnimal
• dmaza

• MSantos

• Majavah
• Legoktm

• Samwilson

• WMDE-Fisch

• mglaser
• Robert Vogel
• ItSpiderman

• Jayprakash12345

• Zabe

• Huji

• Kosta Harlan

• D3r1ck01

• samwilson

• mglaser
• Robert Vogel
• ItSpiderman

• Cicalese
• Robert Vogel

• Lewis Cawte

• Ammarpad

• D3r1ck01

• samwilson

• mglaser
• Robert Vogel
• ItSpiderman

• Gergő Tisza

• Samwilson

• Jsn.sherman
• Scardenasmolinar
• kgraessle

• Cicalese

• WMDE-Fisch

• mglaser
• Robert Vogel
• ItSpiderman

• samwilson

• Cparle

• Ammarpad
• MusikAnimal
• Samwilson

• Catrope

• Gergő Tisza

• Hoo man (only commits where at least one of the changed files match /^client/resources/|^client/includes/DataAccess/|^(client|lib|repo)/includes/UpdateRepo.*/)

• Cparle

• TheDJ
• samtar
• Samwilson
• MusikAnimal

• Alex Paskulin

• Daimona Eaytoy

• Jon Harald Søby

• Samwilson
• Sohom Datta

• mglaser
• Robert Vogel
• ItSpiderman

• Sohom Datta (only commits where at least one of the changed files match /^(modules/|tests/selenium|tests/qunit)/)

• Sebastian Berlin (WMSE)
• Lokal Profil

• Sebastian Berlin (WMSE)
• Lokal Profil

• mglaser
• Robert Vogel
• ItSpiderman

• Robert Vogel
• Dvogel hallowelt

• Robert Vogel
• Dvogel hallowelt

• Robert Vogel

• Robert Vogel

• Robert Vogel

• Samtar

• EBernhardson

• MaxSem
• WMDE-Fisch
• Awight
• dmaza
• Samtar

• Mvolz

• Mvolz

• Physikerwelt

• MSantos

• cscott

• Jdlrobson

• mbsantos

• Ottomata

• Ottomata

• Jack Phoenix
• Jdlrobson

• Dvogel hallowelt

• samwilson

• Paladox

• Alex Paskulin

• Dduvall

• Dduvall

• Kosta Harlan

• DannyS712

• Dduvall
• Gergő Tisza
• ejegg (only commits where at least one of the changed files match /^puppet/modules/(crm|payments|smashpig)/)
• XenoRyet (only commits where at least one of the changed files match /^puppet/modules/(crm|payments|smashpig)/)
• jgleeson (only commits where at least one of the changed files match /^puppet/modules/(crm|payments|smashpig)/)
• cstone (only commits where at least one of the changed files match /^puppet/modules/(crm|payments|smashpig)/)
• Xcollazo (only commits where at least one of the changed files match /^puppet/modules/dumps/)
• jebe (only commits where at least one of the changed files match /^puppet/modules/dumps/)
• milimetric (only commits where at least one of the changed files match /^puppet/modules/dumps/)

• Gergő Tisza (only commits where at least one of the changed files match /^psy/psysh//)

• dcausse

• EBernhardson

• ejegg
• XenoRyet
• jgleeson
• Eileen
• cstone
• damilare
• Wfan

• ejegg
• XenoRyet
• jgleeson
• Eileen
• cstone
• damilare
• Wfan

• ejegg
• XenoRyet
• jgleeson
• Eileen
• cstone
• damilare
• Wfan

• ejegg
• XenoRyet
• jgleeson
• cstone
• damilare
• Wfan

• ejegg
• XenoRyet
• jgleeson
• cstone
• damilare
• Wfan

• cwdent
• ejegg

• cwdent
• ejegg
• XenoRyet
• jgleeson
• cstone
• damilare
• Wfan

• ejegg
• XenoRyet
• jgleeson
• cstone
• damilare
• Wfan

• ejegg

• ejegg
• XenoRyet
• jgleeson
• cstone
• damilare
• Wfan

• ejegg
• XenoRyet
• jgleeson
• cstone
• damilare
• Wfan

• Volans
• Jbond
• dcaro (only commits where at least one of the changed files match /.*wmcs.*/)

• Jbond

• Volans
• Jbond

• Volans
• Giuseppe Lavagetto
• Vgutierrez
• Alexandros Kosiaris

• bd808 (only commits where at least one of the changed files match /^(helmfile\.d/services|charts)/(developer-portal|toolhub)/)
• Urbanecm (only commits where at least one of the changed files match /^(helmfile\.d/services|charts)/linkrecommendation//)

• Giuseppe Lavagetto
• Alexandros Kosiaris
• Hashar

• Ayounsi
• Cathal Mooney

• bd808 (only commits where at least one of the changed files match /^modules/toollabs/|^hieradata/labs/tools//)
• Filippo Giunchedi (only commits where at least one of the changed files match /^.*icinga.*/)
• bblack (only commits where at least one of the changed files match /^modules/(authdns|pybal|lvs|nginx|tlsproxy|varnish|sslcert)/|^modules/role/manifests/(cache|lvs)/|^manifests/role/(authdns|lvs|tlsproxy)\.pp|^files/ssl/|^templates/varnish//)
• Volans (only commits where at least one of the changed files match /(\.py(\.erb)?$|cumin|failoid|debmonitor|spicerack|cookbook|netbox|homer|puppetboard|reimage)/)
• Giuseppe Lavagetto (only commits where at least one of the changed files match /^hieradata/regex.yaml|^modules/(mediawiki|service|httpd|php|etcd)/)
• Muehlenhoff (only commits where at least one of the changed files match /^modules/admin//)
• Jbond (only commits where at least one of the changed files match /modules/admin|apereo_cas|apt|wmflib|modules/base|cumin|cfssl|debian|debdeploy|debmonitor|kerberos|yubiauth|puppet|pki|ssh|sudo|sslcert|sysctl|modules/systemd?|ulogd|ferm|modules/security|standard|wikidough|\.rb$|(Gem|Rake)file|CONTRIBUTORS|^vendor_modules//)
• Slyngshede (only commits where at least one of the changed files match /modules/admin|apereo_cas//)
• Alexandros Kosiaris (only commits where at least one of the changed files match /^modules/(calico|docker|k8s|profile/kubernetes|etherpad|ferm|ganeti|lvs|network|ores|otrs|package_builder|service)/)
• paladox (only commits where at least one of the changed files match /^modules/(gerrit|phabricator)/)
• Xcollazo (only commits where at least one of the changed files match /^modules/(dumps|snapshot)/|^modules/(profile|role)/manifests/dumps//)
• jebe (only commits where at least one of the changed files match /^modules/(dumps|snapshot)/|^modules/(profile|role)/manifests/dumps//)
• Ssingh (only commits where at least one of the changed files match /^modules/(auditd|dnsdist|dnsrecursor)|^modules/(profile|role)/*/(wikidough|durum)/)
• Legoktm (only commits where at least one of the changed files match /^modules/(extdist|codesearch|libraryupgrader|mailman|mailman3)//)
• Cwhite (only commits where at least one of the changed files match /(opensearch|logstash)/)
• dcaro (only commits where at least one of the changed files match /^modules/(toollabs|openstack|ceph)/|^hieradata/labs/tools/|^modules/(profile|role)/*/(toolforge|openstack|toollabs|labs)|modules/base/*/standard_packages//)
• Hoo man (only commits where at least one of the changed files match /^modules/snapshot/(files/systemdjobs/wikibase|manifests/systemdjobs/(wiki(base|data).*\.pp|wikidatadumps/))/)
• Dr0ptp4kt (only commits where at least one of the changed files match /(filtered_tables\.txt|realm\.pp|index-conf\.yaml|maintain-views\.yaml)$/)
• Dzahn (only commits where at least one of the changed files match //^modules/(gerrit|phabricator|wikistats|planet|vrts|etherpad|gitlab|requesttracker|releases|contint|)/|^modules/(profile|role)/manifests/(gerrit|phabricator|planet|wikistats|vrts|etherpad|gitlab|requesttracker|releases|ci|miscweb|microsites|stewards|collaboration_services|aphlict)/)

• Ottomata

• Ottomata

• Ottomata

• Ottomata

• Hoo man (only commits where at least one of the changed files match /^wmf-config/Wikibase\.php/)

• Volans
• Alexandros Kosiaris

• Volans
• Giuseppe Lavagetto
• Alexandros Kosiaris
• CDanis

• Volans
• Jbond

• Volans
• Jbond
• Slyngshede

• Volans
• Jbond

• Volans
• Slyngshede

• Cwhite

• Volans

• Volans
• Ayounsi
• Cathal Mooney
• Jbond

• Volans
• Alexandros Kosiaris

• Volans
• Alexandros Kosiaris
• Jbond

• Volans
• Jbond

• Volans
• Jbond

• Volans
• Jbond

• Volans
• Jbond

• Zhuyifei1999

• Elukey
• Ottomata

• Volans
• Alexandros Kosiaris

• Legoktm
• D3r1ck01

• awight

• Legoktm

• bd808
• Zhuyifei1999

• African Hope

• Dereckson
• Rosalieper

• Samwilson
• Gergő Tisza

• Urbanecm

• Merlijn van Deen

• Samwilson

• Urbanecm

• Urbanecm

• Samwilson

• Urbanecm

• Urbanecm

• Urbanecm

• Urbanecm

• Jean-Frédéric
• Multichill
• Lokal Profil

• Kosta Harlan

• Urbanecm

• Majavah

• Majavah

• Majavah

• MSantos

• MSantos

• MSantos

• MSantos

• Andrew Bogott

• Andrew Bogott

• dbrant

• dbrant
• Cooltey

• D3r1ck01 (only commits where at least one of the changed files match /^scripts/)

• Dachary

• Ladsgroup

• John Vandenberg

• joal

• joal

• zhuyifei1999

• ottomata
• joal
• aqu

• ottomata
• joal
• aqu

• joal

• milimetric
• Nuria
• YuviPanda

• DCausse
• Dr0ptp4kt

• DCausse
• BKing
• Dr0ptp4kt

• Gergő Tisza

• Gergő Tisza

• TJones

• Legoktm

• Jdrewniak
• Mxn

• Takidelfin

• dcausse

• Revi
• Majavah
• Samtar

• Samtar

• MacFan4000

• Ottomata

• fgiunchedi

• SBassett

• Kosta Harlan
• Urbanecm

• bd808
• raymond-ndibe
• sstefanova

• bd808
• Alex Paskulin
• aklapper
• triciaburmeister
• KBach

• Jdlrobson

• Sbisson
• Huei Tan
• Eamedina

• fnegri
• raymond-ndibe
• sstefanova

• fnegri
• raymond-ndibe
• sstefanova

• fnegri
• raymond-ndibe
• sstefanova

• fnegri
• raymond-ndibe
• sstefanova

• Cwhite
• dancy
• jnuche

• Aklapper

• Aklapper

Projects are matched based on unix filename matching as implemented in the fnmatch module. The idea is to set a file_regexp, which filters changesets by changed files instead of repository.

• thcipriani (only commits where at least one of the changed files match /scap/scap\.cfg/)