Open main menu

Git/Reviewers

< Git
Manual on MediaWiki Tools
Crystal Clear action run.png
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.

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.

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.

Contents

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 flags re.I and re.S, and method 'search' (rather than 'match'). 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)$/)
  • Siebrand (only commits where at least one of the changed files match /i18n/)
  • Siebrand (only commits where at least one of the changed files match /Messages/)
  • Siebrand (only commits where at least one of the changed files match /en.json/)
  • Zoranzoki21 (only commits where at least one of the changed files match /sr-ec.json/)
  • Zoranzoki21 (only commits where at least one of the changed files match /sr-el.json/)
  • Zoranzoki21 (only commits where at least one of the changed files match /hr.json/)
  • Zoranzoki21 (only commits where at least one of the changed files match /bs.json/)
  • Thcipriani (only commits where at least one of the changed files match /scap/scap\.cfg/)

integration/*Edit

integration/quibbleEdit

search/*Edit

wikimedia/discovery/*Edit

mediawiki/extensions/BlueSpice*Edit

mediawiki/skins/BlueSpice*Edit

wikimedia-cz/*Edit

Listen to specific repositoriesEdit

cergenEdit

mediawiki/coreEdit

  • Aaron Schulz (only commits where at least one of the changed files match /^(includes/(filebackend|filerepo|job|jobqueue|db|cache|objectcache|externalstore|media|upload|poolcounter)/|maintenance/archive/[a-zA-Z]+)/)
  • Anomie (only commits where at least one of the changed files match /^(tests/phpunit/)?includes/(api|session|auth|libs/Message|Message|libs/ParamValidator|ParamValidator)/|^includes/specials/SpecialBotPasswords\.php$|^includes/user/(BotPassword|LoggedOutEditToken|CentralIdLookup|LocalIdLookup)\.php$|^includes/CommentStore(Comment)?\.php$|^includes/ActorMigration.php$|^includes/language/Message\.php$|^resources/src/mediawiki\.special\.apisandbox|^tests/phpunit/structure/ApiStructureTest\.php$/)
  • Catrope (only commits where at least one of the changed files match /includes/resourceloader|CSSMin|JavaScriptMinifier|resources/src/startup/|resources/src/mediawiki.base/|SpecialJavaScriptTest/)
  • cscott (only commits where at least one of the changed files match /[Pp]arser|virtualrest/)
  • 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//)
  • mstyles (only commits where at least one of the changed files match /^includes/search//)
  • dmaza (only commits where at least one of the changed files match /^includes/.*block/)
  • 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$/)
  • Krinkle (only commits where at least one of the changed files match /includes/resourceloader|CSSMin|JavaScriptMinifier|resources/src/startup/|resources/src/mediawiki.base/|SpecialJavaScriptTest/)
  • Legoktm (only commits where at least one of the changed files match /^includes/(config|registration)/)
  • Liangent (only commits where at least one of the changed files match /^includes/ZhConversion\.php|^maintenance/language/zhtable/|^languages/LanguageConverter\.php|^languages/classes/LanguageZh/)
  • Parent5446 (only commits where at least one of the changed files match /^(maintenance|includes/(db|password|Hooks\.php|Timestamp.php))/)
  • Skizzerz (only commits where at least one of the changed files match /[Mm]ssql/)
  • Smalyshev (only commits where at least one of the changed files match /^includes/search//)
  • 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|interwiki|Tag/)
  • Zppix (only commits where at least one of the changed files match /languages/l18n/en.json/)

mediawiki/debianEdit

mediawiki/event-schemasEdit

mediawiki/extensions/AbuseFilterEdit

mediawiki/extensions/ActiveAbstractEdit

mediawiki/extensions/AdvancedSearchEdit

mediawiki/extensions/AnnotatorEdit

mediawiki/extensions/ApiFeatureUsageEdit

mediawiki/extensions/BlogPageEdit

mediawiki/extensions/CampaignsEdit

mediawiki/extensions/CapiuntoEdit

mediawiki/extensions/CirrusSearchEdit

mediawiki/extensions/CiteEdit

mediawiki/extensions/CentralAuthEdit

  • Anomie (only commits where at least one of the changed files match /^includes/session//)

mediawiki/extensions/CentralNoticeEdit

mediawiki/extensions/CheckUserEdit

mediawiki/extensions/CodeEditorEdit

mediawiki/extensions/CodeReviewEdit

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/CreateRedirectEdit

mediawiki/extensions/CreateUserPageEdit

mediawiki/extensions/Description2Edit

mediawiki/extensions/DisplayTitleEdit

mediawiki/extensions/DonationInterfaceEdit

mediawiki/extensions/EchoEdit

mediawiki/extensions/EUCopyrightCampaignEdit

mediawiki/extensions/EducationProgramEdit

mediawiki/extensions/ElectronPdfServiceEdit

mediawiki/extensions/EmailAuthorizationEdit

mediawiki/extensions/EventBusEdit

mediawiki/extensions/EventLoggingEdit

mediawiki/extensions/ExtensionDistributorEdit

mediawiki/extensions/FileExporterEdit

mediawiki/extensions/FileImporterEdit

mediawiki/extensions/FlaggedRevsEdit

mediawiki/extensions/FlowEdit

mediawiki/extensions/GadgetsEdit

  • MaxSem
  • MarcoAurelio (only commits where at least one of the changed files match /en.json/)
  • MarcoAurelio (only commits where at least one of the changed files match /Gadgets\.(?:alias|namespaces)\.php/)
  • DannyS712

mediawiki/extensions/GenealogyEdit

mediawiki/extensions/GeoDataEdit

mediawiki/extensions/GettingStartedEdit

mediawiki/extensions/GlobalCssJsEdit

mediawiki/extensions/GlobalCheckUserEdit

mediawiki/extensions/GlobalPreferencesEdit

mediawiki/extensions/GlobalUserPageEdit

mediawiki/extensions/GraphVizEdit

mediawiki/extensions/GrowthExperimentsEdit

mediawiki/extensions/GuidedTourEdit

mediawiki/extensions/GuidedTour/guidersEdit

mediawiki/extensions/HierarchyBuilderEdit

mediawiki/extensions/IdsEdit

mediawiki/extensions/InputBoxEdit

mediawiki/extensions/JSBreadCrumbsEdit

mediawiki/extensions/JsonConfigEdit

mediawiki/extensions/KartographerEdit

mediawiki/extensions/LabeledSectionTransclusionEdit

mediawiki/extensions/LDAPAuthentication2Edit

mediawiki/extensions/LDAPAuthorizationEdit

mediawiki/extensions/LingoEdit

mediawiki/extensions/LinterEdit

mediawiki/extensions/LoginNotifyEdit

mediawiki/extensions/MachineVisionEdit

mediawiki/extensions/MathEdit

mediawiki/extensions/MathSearchEdit

mediawiki/extensions/MassMessageEdit

mediawiki/extensions/MediaWikiAuthEdit

mediawiki/extensions/MediaWikiChatEdit

mediawiki/extensions/MediaWikiFarmEdit

mediawiki/extensions/MobileAppEdit

mediawiki/extensions/MobileFrontendEdit

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

mediawiki/extensions/TextExtractsEdit

mediawiki/extensions/MolHandlerEdit

mediawiki/extensions/NamespacePathsEdit

mediawiki/extensions/NSFileRepoEdit

mediawiki/extensions/OpenBadgesEdit

mediawiki/extensions/OpenIDConnectEdit

mediawiki/extensions/ORESEdit

mediawiki/extensions/ParserFunctionsEdit

mediawiki/extensions/PluggableAuthEdit

mediawiki/extensions/PopupsEdit

mediawiki/extensions/QuizEdit

mediawiki/extensions/RandomInCategoryEdit

mediawiki/extensions/ReadingListsEdit

mediawiki/extensions/RelatedArticlesEdit

mediawiki/extensions/ReplaceSetEdit

mediawiki/extensions/RevisionSliderEdit

mediawiki/extensions/ScribuntoEdit

  • Anomie (only commits where at least one of the changed files match /^(?!Scribunto\.(?:magic|namespaces)\.php$).*/)
  • Hoo man (only commits where at least one of the changed files match /^engines/LuaCommon/(lualib/mw.html.lua|HtmlLibrary.php)|^tests/engines/LuaCommon/HtmlLibraryTest.*/)
  • Jackmcbarn
  • MarcoAurelio (only commits where at least one of the changed files match /en.json/)
  • MarcoAurelio (only commits where at least one of the changed files match /Scribunto\.(?:magic|namespaces)\.php/)

mediawiki/extensions/SecureAuthEdit

mediawiki/extensions/SecureLinkFixerEdit

mediawiki/extensions/SecurePollEdit

mediawiki/extensions/SendGridEdit

mediawiki/extensions/SidebarDonateBoxEdit

mediawiki/extensions/SimpleSAMLphpEdit

mediawiki/extensions/SocialProfileEdit

mediawiki/extensions/SpamBlacklistEdit

mediawiki/extensions/SparkPostEdit

mediawiki/extensions/SudoEdit

mediawiki/extensions/TemplateSandboxEdit

mediawiki/extensions/TemplateStylesEdit

mediawiki/extensions/TemplateWizardEdit

mediawiki/extensions/ThemeDesignerEdit

mediawiki/extensions/ThrottleOverrideEdit

mediawiki/extensions/TitleBlacklistEdit

mediawiki/extensions/TitleIconEdit

mediawiki/extensions/TwoColConflictEdit

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.*/)
  • Smalyshev (only commits where at least one of the changed files match /^repo/includes/Search//)

mediawiki/extensions/WikibaseMediaInfoEdit

mediawiki/extensions/WikiEditorEdit

mediawiki/extensions/WikiForumEdit

mediawiki/extensions/WikispeechEdit

mediawiki/extensions/WikimediaEditorTasksEdit

mediawiki/extensions/WikimediaMaintenanceEdit

mediawiki/extensions/WikimediaMessagesEdit

mediawiki/libs/phpstorm-stubsEdit

mediawiki/php/luasandboxEdit

mediawiki/php/wikidiff2Edit

mediawiki/services/citoidEdit

mediawiki/services/citoid/deployEdit

mediawiki/services/mathoidEdit

mediawiki/services/mathoid/deployEdit

mediawiki/services/mobileappsEdit

mediawiki/services/mobileapps/deployEdit

mediawiki/services/ocg-collectionEdit

mediawiki/services/restbase/deployEdit

mediawiki/services/trending-editsEdit

mediawiki/services/trending-edits/deployEdit

mediawiki/services/wikifeedsEdit

mediawiki/services/eventstreamsEdit

mediawiki/services/eventstreams/deployEdit

mediawiki/skinsEdit

mediawiki/skins/BlueSpiceSkinEdit

mediawiki/skins/EUCopyrightCampaignSkinEdit

mediawiki/skins/MetrolookEdit

mediawiki/skins/MinervaNeueEdit

mediawiki/skins/VectorEdit

mediawiki/ruby/apiEdit

mediawiki/seleniumEdit

mediawiki/tools/codesnifferEdit

mediawiki/tools/releaseEdit

  • Florianschmidtwelzow (only commits where at least one of the changed files match /^make-deploy-notes/)

mediawiki/vagrantEdit

  • Dduvall
  • BryanDavis
  • ejegg (only commits where at least one of the changed files match /^puppet/modules/(crm|payments|smashpig)/)
  • mepps (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)/)
  • Anomie (only commits where at least one of the changed files match /^puppet/modules/role/templates/oauth/hello_world.php.erb/)
  • ArielGlenn (only commits where at least one of the changed files match /^puppet/modules/dumps/)

test/mediawiki/extensions/examplesEdit

  • Merlijn van Deen (only commits where at least one of the changed files match /^test\.txt$/)

wikimedia/communications/WMBlogEdit

wikimedia/communications/WP-VictorEdit

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

operations/debs/debdeployEdit

operations/debs/pybalEdit

operations/docker-images/docker-pkgEdit

operations/homer/publicEdit

operations/puppetEdit

  • Merlijn van Deen (only commits where at least one of the changed files match /^modules/toollabs/|^hieradata/labs/tools//)
  • BryanDavis (only commits where at least one of the changed files match /^modules/toollabs/|^hieradata/labs/tools//)
  • Hashar (only commits where at least one of the changed files match /^modules/(beta|contint|jenkins|nodepool|zuul)/|^module/(profile|role)/manifests/(beta|ci|gerrit|nodepool|zuul)/)
  • 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)/)
  • fgiunchedi (only commits where at least one of the changed files match /^(files|manifests|templates)/swift/|^manifests/role/swift/)
  • 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//)
  • Krenair (only commits where at least one of the changed files match /^modules/(mediawiki|admin|openstack)//)
  • Gehel (only commits where at least one of the changed files match /^modules/(base|elasticsearch|kartotherian|logstash|tilerator|wdqs)//)
  • Mholloway (only commits where at least one of the changed files match /^modules/mobileapps//)
  • ema (only commits where at least one of the changed files match /^modules/(lvs|nginx|tlsproxy|varnish)/|^modules/role/manifests/(cache|lvs)/|^manifests/role/(lvs|tlsproxy)\.pp|^files/ssl/|^templates/varnish//)
  • mobrovac (only commits where at least one of the changed files match /^modules/(service|citoid|changeprop|graphoid|mathoid|mobileapps|cxserver|restbase|parsoid)//)
  • 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|ulogd|ferm|security|standard)/|^modules/(profile|role)/*/(debdeploy|debmonitor|kerberos|yubiauth|base)/)
  • Smalyshev (only commits where at least one of the changed files match /^modules/wdqs//)
  • Alexandros Kosiaris (only commits where at least one of the changed files match /^modules/(backup|bacula|calico|docker|k8s|profile/kubernetes|etherpad|ferm|ganeti|lvs|network|ores|otrs|osm|package_builder|service)/)
  • paladox (only commits where at least one of the changed files match /^modules/gerrit/)
  • ArielGlenn (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)//)

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/(abusefilter|Wikibase.*)\.php/)
  • aude (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/external-monitoringEdit

operations/software/homerEdit

operations/software/keyholderEdit

operations/software/netboxEdit

operations/software/netbox-deployEdit

operations/software/netbox-reportsEdit

operations/software/spicerackEdit

operations/software/tools-webserviceEdit

operations/software/varnish/varnishkafkaEdit

operations/switchdcEdit

labs/codesearchEdit

labs/icinga2Edit

labs/toollabsEdit

labs/tools/awmd-statsEdit

labs/tools/Commons-twitter-botEdit

labs/tools/watch-translationsEdit

labs/tools/wikibugs2Edit

labs/tools/grrritEdit

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/urbanecmbotEdit

labs/tools/wikipedia-android-buildsEdit

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/bots/CommonsDelinkerEdit

pywikibot/coreEdit

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

pywikibot/bots/FLOSSbotEdit

pywikibot/spellingEdit

pywikibot/i18nEdit

pywikibot/compatEdit

analytics/quarry/webEdit

analytics/refineryEdit

analytics/refinery/sourceEdit

analytics/limn-mobile-dataEdit

wikidata/query/rdfEdit

wikidata/query/deployEdit

wikidata/query/guiEdit

wikidata/query/gui-deployEdit

wikimedia/textcatEdit

css-sanitizerEdit

php-session-serializerEdit

integration/commit-message-validatorEdit

wikimedia/portalsEdit

wikimedia-cz/trackerEdit

labs/tools/stewardbotsEdit

labs/tools/ZppixBotEdit

eventloggingEdit

wmf-utilsEdit

mapdataEdit

wikimedia/security/toolingEdit