Git/Reviewers

< Git

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

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 Note: If you are looking for a list of people that could review your patch, please see Developers/Maintainers.

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.

How to add yourselfEdit

Each project has a header with the Gerrit project name, followed by a list of people (using Template:Gerrit-reviewer), 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.

Listen to repository groupsEdit

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.

*Edit

  • Jackmcbarn (only commits where at least one of the changed files match /([Ll]ibrary\.php|\.lua)$/)
  • 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/)

cloud/metricsinfra/*Edit

integration/*Edit

integration/quibbleEdit

search/*Edit

wikimedia/discovery/*Edit

mediawiki/extensions/BlueSpice*Edit

mediawiki/skins/BlueSpice*Edit

mediawiki/tools/phan*Edit

wikimedia-cz/*Edit

mediawiki/services/wikispeech*Edit

cloud/toolforge/*Edit


Listen to specific repositoriesEdit

cergenEdit

integration/tox-wikimediaEdit

mediawiki/coreEdit

  • 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]+)/)
  • ArielGlenn (only commits where at least one of the changed files match /^includes/export/|^maintenance/includes/(Backup|TextPass|SevenZip)|^maintenance/(fetchText|dump)/)
  • Hokwelum (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 /includes/ResourceLoader|resources/src/startup/|resources/src/mediawiki.base/|resources/src/vue/|SpecialJavaScriptTest/)
  • 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//)
  • DannyS712 (only commits where at least one of the changed files match /(MediaWikiServices|[Uu]ser|[Bb]lock|ResourceLoader|resources/src/startup/|resources/src/mediawiki\.base/)/)
  • 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)/)
  • Jackmcbarn (only commits where at least one of the changed files match /(?<!^includes/Template)[Pp]arse|^includes/content/|^includes/(EditPage|Linker|MagicWord|Sanitizer|api/ApiExpandTemplates).php$/)
  • Jforrester (only commits where at least one of the changed files match /oojs/)
  • 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/)
  • Krinkle (only commits where at least one of the changed files match /includes/ResourceLoader|resources/src/startup/|resources/src/mediawiki.base/|SpecialJavaScriptTest/)
  • 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/)
  • 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)/)
  • TTO (only commits where at least one of the changed files match /Import/)
  • 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/)

mediawiki/event-schemasEdit

mediawiki/extensions/AbuseFilterEdit

mediawiki/extensions/ActiveAbstractEdit

mediawiki/extensions/AdvancedSearchEdit

mediawiki/extensions/AdvancedMetaEdit

mediawiki/extensions/AnnotatorEdit

mediawiki/extensions/BlogPageEdit

mediawiki/extensions/CampaignEventsEdit

mediawiki/extensions/CampaignsEdit

mediawiki/extensions/CapiuntoEdit

mediawiki/extensions/CirrusSearchEdit

mediawiki/extensions/CiteEdit

mediawiki/extensions/CentralAuthEdit

mediawiki/extensions/CentralNoticeEdit

mediawiki/extensions/ChessBrowserEdit

mediawiki/extensions/CheckUserEdit

mediawiki/extensions/CodeEditorEdit

mediawiki/extensions/CodeReviewEdit

mediawiki/extensions/CognitiveProcessDesignerEdit

mediawiki/extensions/CollectionEdit

mediawiki/extensions/Collection/OfflineContentGeneratorEdit

mediawiki/extensions/Collection/OfflineContentGenerator/bundlerEdit

mediawiki/extensions/Collection/OfflineContentGenerator/latex_rendererEdit

mediawiki/extensions/CommentsEdit

mediawiki/extensions/CommentStreamsEdit

mediawiki/extensions/ContactPageEdit

mediawiki/extensions/ContainerFilterEdit

mediawiki/extensions/ContentDropletsEdit

mediawiki/extensions/ContentTransferEdit

mediawiki/extensions/ContentStabilizationEdit

mediawiki/extensions/CreateUserPageEdit

mediawiki/extensions/DarkModeEdit

mediawiki/extensions/DisplayTitleEdit

mediawiki/extensions/DonationInterfaceEdit

mediawiki/extensions/DrawioEditorEdit

mediawiki/extensions/EchoEdit

mediawiki/extensions/EUCopyrightCampaignEdit

mediawiki/extensions/EducationProgramEdit

mediawiki/extensions/ElectronPdfServiceEdit

mediawiki/extensions/EmailAuthorizationEdit

mediawiki/extensions/EventBusEdit

mediawiki/extensions/FileExporterEdit

mediawiki/extensions/FileImporterEdit

mediawiki/extensions/FlaggedRevsEdit

mediawiki/extensions/FormsEdit

mediawiki/extensions/GadgetsEdit

mediawiki/extensions/GenealogyEdit

mediawiki/extensions/GeoDataEdit

mediawiki/extensions/GettingStartedEdit

mediawiki/extensions/GlobalPreferencesEdit

mediawiki/extensions/GlobalWatchlistEdit

mediawiki/extensions/GraphVizEdit

mediawiki/extensions/GrowthExperimentsEdit

  • Kosta Harlan
  • Gergő Tisza
  • Sergio Gimeno
  • KMorgan
  • Urbanecm (only commits where at least one of the changed files match /^includes/(Config|Mentorship|MentorDashboard|PeriodicMetrics|Specials/(Forms|SpecialClaimMentee|SpecialEditGrowthConfig|SpecialEnrollAsMentor|SpecialMentorDashboard|SpecialQuitMentorship))/)

mediawiki/extensions/GuidedTourEdit

mediawiki/extensions/GuidedTour/guidersEdit

mediawiki/extensions/HierarchyBuilderEdit

mediawiki/extensions/IdsEdit

mediawiki/extensions/ImageMapEditEdit

mediawiki/extensions/InputBoxEdit

mediawiki/extensions/JSBreadCrumbsEdit

mediawiki/extensions/KartographerEdit

mediawiki/extensions/LabeledSectionTransclusionEdit

mediawiki/extensions/LDAPAuthentication2Edit

mediawiki/extensions/LDAPAuthorizationEdit

mediawiki/extensions/LDAPGroupsEdit

mediawiki/extensions/LDAPProviderEdit

mediawiki/extensions/LDAPUserInfoEdit

mediawiki/extensions/LingoEdit

mediawiki/extensions/LoginNotifyEdit

mediawiki/extensions/MachineVisionEdit

mediawiki/extensions/MathEdit

mediawiki/extensions/MathSearchEdit

mediawiki/extensions/MassMessageEdit

mediawiki/extensions/MediaUploaderEdit

mediawiki/extensions/MediaWikiChatEdit

mediawiki/extensions/MediaWikiFarmEdit

mediawiki/extensions/MediaSearchEdit

mediawiki/extensions/MissedPagesEdit

mediawiki/extensions/MobileAppEdit

mediawiki/extensions/MobileFrontendEdit

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

mediawiki/extensions/MolHandlerEdit

mediawiki/extensions/NSFileRepoEdit

mediawiki/extensions/OpenBadgesEdit

mediawiki/extensions/OpenIDConnectEdit

mediawiki/extensions/PageCheckoutEdit

mediawiki/extensions/PageHeaderEdit

mediawiki/extensions/PageTriageEdit

mediawiki/extensions/ParserFunctionsEdit

mediawiki/extensions/PluggableAuthEdit

mediawiki/extensions/PhonosEdit

mediawiki/extensions/ReadingListsEdit

mediawiki/extensions/RealMeEdit

mediawiki/extensions/RedirectManagerEdit

mediawiki/extensions/RevisionSliderEdit

mediawiki/extensions/ScribuntoEdit

mediawiki/extensions/SectionAnchorsEdit

mediawiki/extensions/SecureAuthEdit

mediawiki/extensions/SecureLinkFixerEdit

mediawiki/extensions/SecurePollEdit

mediawiki/extensions/SendGridEdit

mediawiki/extensions/ShareEdit

mediawiki/extensions/SimpleCalendarEdit

mediawiki/extensions/SimpleSAMLphpEdit

mediawiki/extensions/SocialProfileEdit

mediawiki/extensions/SpamBlacklistEdit

mediawiki/extensions/SparkPostEdit

mediawiki/extensions/SpecialNamespacesEdit

mediawiki/extensions/StandardDialogsEdit

mediawiki/extensions/TemplateDataEdit

mediawiki/extensions/TemplateSandboxEdit

mediawiki/extensions/TemplateWizardEdit

mediawiki/extensions/TitleBlacklistEdit

mediawiki/extensions/TitleIconEdit

mediawiki/extensions/TwoColConflictEdit

mediawiki/extensions/UnifiedTaskOverviewEdit

mediawiki/extensions/UnlinkedWikibaseEdit

mediawiki/extensions/UploadWizardEdit

mediawiki/extensions/UrlShortenerEdit

mediawiki/extensions/VueTestEdit

mediawiki/extensions/WikibaseEdit

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

mediawiki/extensions/WikibaseMediaInfoEdit

mediawiki/extensions/WikiEditorEdit

mediawiki/extensions/WikiForumEdit

mediawiki/extensions/WikimediaApiPortalOAuthEdit

mediawiki/extensions/WikimediaIncubatorEdit

mediawiki/extensions/WikisourceEdit

mediawiki/extensions/PreToClipEdit

mediawiki/extensions/ProofreadPageEdit

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

mediawiki/extensions/WikispeechEdit

mediawiki/extensions/WikispeechSpeechDataCollectorEdit

mediawiki/extensions/WikimediaEventsEdit

mediawiki/extensions/WorkflowsEdit

mediawiki/libs/IPAValidatorEdit

mediawiki/libs/phpstorm-stubsEdit

mediawiki/php/luasandboxEdit

mediawiki/php/wikidiff2Edit

mediawiki/services/citoidEdit

mediawiki/services/citoid/deployEdit

mediawiki/services/mathoidEdit

mediawiki/services/mobileappsEdit

mediawiki/services/ocg-collectionEdit

mediawiki/services/trending-editsEdit

mediawiki/services/wikifeedsEdit

mediawiki/services/eventstreamsEdit

mediawiki/services/eventstreams/deployEdit

mediawiki/skinsEdit

mediawiki/skins/BlueSpiceSkinEdit

mediawiki/skins/EUCopyrightCampaignSkinEdit

mediawiki/skins/ForegroundEdit

mediawiki/skins/MetrolookEdit

mediawiki/skins/MinervaNeueEdit

mediawiki/skins/VectorEdit

mediawiki/skins/WikimediaApiPortalEdit

mediawiki/ruby/apiEdit

mediawiki/seleniumEdit

mediawiki/tools/cliEdit

mediawiki/tools/codesnifferEdit

mediawiki/vagrantEdit

  • 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)/)
  • ArielGlenn (only commits where at least one of the changed files match /^puppet/modules/dumps/)
  • Hokwelum (only commits where at least one of the changed files match /^puppet/modules/dumps/)

mediawiki/vendorEdit

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

wikimedia/discovery/relevanceForgeEdit

wikimedia/discovery/discernatronEdit

wikimedia/fundraising/crmEdit

wikimedia/fundraising/crm/civicrmEdit

wikimedia/fundraising/crm/drupalEdit

wikimedia/fundraising/dashEdit

wikimedia/fundraising/FRUECEdit

wikimedia/fundraising/php-queueEdit

wikimedia/fundraising/process-controlEdit

wikimedia/fundraising/SmashPigEdit

wikimedia/fundraising/statsEdit

wikimedia/fundraising/toolsEdit

wikimedia/fundraising/tools/DjangoBannerStatsEdit

operations/cookbooksEdit

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

operations/debs/wmf-certificatesEdit

operations/debs/debdeployEdit

operations/debs/pybalEdit

operations/deployment-chartsEdit

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

operations/docker-images/docker-pkgEdit

operations/homer/publicEdit

operations/puppetEdit

  • bd808 (only commits where at least one of the changed files match /^modules/toollabs/|^hieradata/labs/tools//)
  • Dzahn (only commits where at least one of the changed files match /^modules/(gerrit|phabricator|bugzilla_static|wikistats|planet)/|^module/(profile|role)/manifests/(gerrit|phabricator|bugzilla_static|planet)/)
  • 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//)
  • 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)/)
  • ArielGlenn (only commits where at least one of the changed files match /^modules/(dumps|snapshot)/|^modules/(profile|role)/manifests/dumps//)
  • Hokwelum (only commits where at least one of the changed files match /^modules/(dumps|snapshot)/|^modules/(profile|role)/manifests/dumps//)
  • Arturo Borrero Gonzalez (only commits where at least one of the changed files match /^modules/(toollabs|openstack)/|^hieradata/labs/tools/|^modules/(profile|role)/*/(toolforge|openstack|toollabs|labs)|modules/base/*/standard_packages//)
  • 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/))/)

operations/puppet/cdhEdit

operations/puppet/cdh4Edit

operations/puppet/kafkaEdit

operations/puppet/zookeeperEdit

operations/mediawiki-configEdit

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

operations/softwareEdit

operations/software/conftoolEdit

operations/software/cuminEdit

operations/software/debmonitorEdit

operations/software/debmonitor/deployEdit

operations/software/ecsEdit

operations/software/external-monitoringEdit

operations/software/homerEdit

operations/software/keyholderEdit

operations/software/netboxEdit

operations/software/netbox-deployEdit

operations/software/netbox-extrasEdit

operations/software/pywmflibEdit

operations/software/spicerackEdit

operations/software/tools-webserviceEdit

operations/software/varnish/varnishkafkaEdit

operations/switchdcEdit

labs/codesearchEdit

labs/icinga2Edit

labs/libraryupgraderEdit

labs/toollabsEdit

labs/tools/awmd-statsEdit

labs/tools/Commons-twitter-botEdit

labs/tools/extjsonuploaderEdit

labs/tools/watch-translationsEdit

labs/tools/wikibugs2Edit

labs/tools/wikisource-ocrEdit

labs/tools/map-of-monumentsEdit

labs/tools/massmailerEdit

labs/tools/wikinityEdit

labs/tools/weapon-of-mass-descriptionEdit

labs/tools/commons-mass-descriptionEdit

labs/tools/harvesting-data-refineryEdit

labs/tools/heritageEdit

labs/tools/sonarqubebotEdit

labs/tools/urbanecmbotEdit

labs/tools/train-blockersEdit

labs/tools/github-pr-closerEdit

labs/tools/majavah-botEdit

maps/kartotherian/packageEdit

maps/kartotherian/deployEdit

maps/tilerator/packageEdit

maps/tilerator/deployEdit

sink_nova_ldapEdit

sink_nova_fixed_multiEdit

apps/android/java-mwapiEdit

apps/android/wikipediaEdit

pywikibot/coreEdit

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

pywikibot/bots/FLOSSbotEdit

pywikibot/spellingEdit

pywikibot/compatEdit

analytics/aqsEdit

analytics/gobblin-wmfEdit

analytics/quarry/webEdit

analytics/refineryEdit

analytics/refinery/sourceEdit

analytics/wikihadoopEdit

analytics/limn-mobile-dataEdit

wikidata/query/rdfEdit

wikidata/query/deployEdit

wikidata/query/guiEdit

wikidata/query/gui-deployEdit

wikidata/query/flink-swift-pluginEdit

wikimedia/textcatEdit

integration/commit-message-validatorEdit

wikimedia/portalsEdit

wikimedia-cz/trackerEdit

labs/tools/stewardbotsEdit

labs/tools/ZppixBotEdit

eventloggingEdit

wmf-utilsEdit

wikimedia/security/toolingEdit

research/mwaddlinkEdit

wikimedia/toolhubEdit

design/codexEdit

wikimedia/developer-portalEdit

mediawiki/extensions/WikistoriesEdit

cloud/toolforge/toolforge-cliEdit

cloud/toolforge/toolforge-build-cliEdit

cloud/toolforge/toolforge-build-apiEdit

cloud/toolforge/buildpacksEdit

releng/phatalityEdit