[MediaWiki-commits] [Gerrit] mediawiki/core[master]: rdbms: Simplify Database::factory()

2018-01-24 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/406063 ) Change subject: rdbms: Simplify Database::factory() .. rdbms: Simplify Database::factory() Follows-up 0ff2b7a776 (T120333) which removed su

[MediaWiki-commits] [Gerrit] CLDRPluralRuleParser[master]: tests: Update phpunit and enable coverage for all classes

2018-01-24 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/406055 ) Change subject: tests: Update phpunit and enable coverage for all classes .. tests: Update phpunit and enable coverage for all classes The

[MediaWiki-commits] [Gerrit] generator-wikimedia-php-library[master]: Archive repository

2018-01-24 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/405929 ) Change subject: Archive repository .. Archive repository Bug: T174416 Change-Id: I0a4a291607da3acf6ba4de7da140c3d8fd4117af --- D .g

[MediaWiki-commits] [Gerrit] integration/docroot[master]: cover: Add sort=cover parameter to coverage report

2018-01-23 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/405996 ) Change subject: cover: Add sort=cover parameter to coverage report .. cover: Add sort=cover parameter to coverage report When enabled, the

[MediaWiki-commits] [Gerrit] integration/docroot[master]: cover: Convert coverage sub-nav from breadcrumbs to nav-tabs

2018-01-23 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/405995 ) Change subject: cover: Convert coverage sub-nav from breadcrumbs to nav-tabs .. cover: Convert coverage sub-nav from breadcrumbs to nav-tabs

[MediaWiki-commits] [Gerrit] integration/config[master]: Remove jobs for 'generator-wikimedia-php-library'

2018-01-23 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/405930 ) Change subject: Remove jobs for 'generator-wikimedia-php-library' .. Remove jobs for 'generator-wikimedia-php-library' Repo is being archiv

[MediaWiki-commits] [Gerrit] generator-wikimedia-php-library[master]: Archive repository

2018-01-23 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/405929 ) Change subject: Archive repository .. Archive repository Bug: T174416 Change-Id: I0a4a291607da3acf6ba4de7da140c3d8fd4117af --- D .gitignore

[MediaWiki-commits] [Gerrit] operations/puppet[production]: webperf: Introduce 'templates' in test fixture and use for m...

2018-01-12 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/404046 ) Change subject: webperf: Introduce 'templates' in test fixture and use for mwload .. webperf: Introduce 'templates' in test fixture and use

[MediaWiki-commits] [Gerrit] operations/puppet[production]: webperf: Re-use expected result by reference to simplify fix...

2018-01-12 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/404045 ) Change subject: webperf: Re-use expected result by reference to simplify fixture .. webperf: Re-use expected result by reference to simplify

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: config: Increase coverage of EtcdConfig::parseDirectory()

2018-01-12 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403980 ) Change subject: config: Increase coverage of EtcdConfig::parseDirectory() .. config: Increase coverage of EtcdConfig::parseDirectory() One

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Initial profiler for Beta Cluster

2018-01-12 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403974 ) Change subject: Initial profiler for Beta Cluster .. Initial profiler for Beta Cluster * profiler-labs.php: Like profiler.php, but for labs

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Remove unused PhpAutoPrepend.php file for now.

2018-01-12 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403973 ) Change subject: Remove unused PhpAutoPrepend.php file for now. .. Remove unused PhpAutoPrepend.php file for now. Follows-up 96de34769, whic

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: JavaScriptMinifier: Remove support for unused $maxLineLength...

2018-01-11 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403835 ) Change subject: JavaScriptMinifier: Remove support for unused $maxLineLength param .. JavaScriptMinifier: Remove support for unused $maxLin

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: JavaScriptMinifier: Enable phpcs and fix violations

2018-01-11 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403769 ) Change subject: JavaScriptMinifier: Enable phpcs and fix violations .. JavaScriptMinifier: Enable phpcs and fix violations Found 90 violati

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: keys: Simplify and update keys.html styling to match other s...

2018-01-10 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403569 ) Change subject: keys: Simplify and update keys.html styling to match other simple pages .. keys: Simplify and update keys.html styling to m

[MediaWiki-commits] [Gerrit] operations/puppet[production]: [WIP] coal: Consume EventLogging from Kafka instead of ZMQ

2018-01-10 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403560 ) Change subject: [WIP] coal: Consume EventLogging from Kafka instead of ZMQ .. [WIP] coal: Consume EventLogging from Kafka instead of ZMQ Bu

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: JavaScriptMinifier: Improve docs for parsing of string literals

2018-01-09 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403331 ) Change subject: JavaScriptMinifier: Improve docs for parsing of string literals .. JavaScriptMinifier: Improve docs for parsing of string li

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: JavaScriptMinifier: Fix "Uninitialized offset" in string and...

2018-01-09 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/403332 ) Change subject: JavaScriptMinifier: Fix "Uninitialized offset" in string and regexp parsing .. JavaScriptMinifier: Fix "Uninitialized offse

[MediaWiki-commits] [Gerrit] mediawiki...Gadgets[master]: tests: Add @covers tags

2018-01-04 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/402145 ) Change subject: tests: Add @covers tags .. tests: Add @covers tags Change-Id: Ia3e18e7149292cd23765636111acc846564bf6cc --- M tests/phpunit

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: phpunit: Remove outdated comment about calling of LinkCache:...

2017-12-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/400434 ) Change subject: phpunit: Remove outdated comment about calling of LinkCache::clear() .. phpunit: Remove outdated comment about calling of L

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: maintenance: Add unit test for Benchmarker class

2017-12-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/400432 ) Change subject: maintenance: Add unit test for Benchmarker class .. maintenance: Add unit test for Benchmarker class Change-Id: I3f835fc077

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: JavaScriptMinifier: Improve docs around parsing of regexp li...

2017-12-22 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/399861 ) Change subject: JavaScriptMinifier: Improve docs around parsing of regexp literals .. JavaScriptMinifier: Improve docs around parsing of re

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: JavaScriptMinifier: Remove support for unused $statementsOnO...

2017-12-22 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/399855 ) Change subject: JavaScriptMinifier: Remove support for unused $statementsOnOwnLine flag .. JavaScriptMinifier: Remove support for unused $s

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: resourceloader: Improve JavaScriptMinifier code coverage

2017-12-22 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/399853 ) Change subject: resourceloader: Improve JavaScriptMinifier code coverage .. resourceloader: Improve JavaScriptMinifier code coverage Change

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: build: Remove phplint --ignore-fails hack for symlink support

2017-12-14 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/398408 ) Change subject: build: Remove phplint --ignore-fails hack for symlink support .. build: Remove phplint --ignore-fails hack for symlink suppo

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: mediawiki.loader: Fix logError() not to print exceptions twice

2017-12-14 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/398319 ) Change subject: mediawiki.loader: Fix logError() not to print exceptions twice .. mediawiki.loader: Fix logError() not to print exceptions t

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: mediawiki.loader: Avoid jQuery for simple creation and selec...

2017-12-13 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/398109 ) Change subject: mediawiki.loader: Avoid jQuery for simple creation and selection .. mediawiki.loader: Avoid jQuery for simple creation and s

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: resourceloader: Use document.head instead of getElementsByTa...

2017-12-13 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/398108 ) Change subject: resourceloader: Use document.head instead of getElementsByTagName .. resourceloader: Use document.head instead of getElemen

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: [WIP] ImagePage: Create metadata table collapse button in PHP

2017-12-04 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395159 ) Change subject: [WIP] ImagePage: Create metadata table collapse button in PHP .. [WIP] ImagePage: Create metadata table collapse button in P

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: ImagePage: Clean up metadata.js collapsing

2017-12-04 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395157 ) Change subject: ImagePage: Clean up metadata.js collapsing .. ImagePage: Clean up metadata.js collapsing In preparation for trying out a CS

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: ImagePage: Make metadata table's initial collapse CSS-only

2017-12-04 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395158 ) Change subject: ImagePage: Make metadata table's initial collapse CSS-only .. ImagePage: Make metadata table's initial collapse CSS-only Ma

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Re-apply "Revert "Set wgCommentTableSchemaMigrationStage = M...

2017-12-04 Thread Krinkle (Code Review)
Hello Urbanecm, Legoktm, jenkins-bot, Anomie, Zoranzoki21, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/395150 to review the following change. Change subject: Re-apply "Revert "Set wgCommentTableSchemaMigrationStage = MIGRATION_WRITE_BOTH on test wikis""

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Revert "Revert "Set wgCommentTableSchemaMigrationStage = MIG...

2017-12-04 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395149 ) Change subject: Revert "Revert "Set wgCommentTableSchemaMigrationStage = MIGRATION_WRITE_BOTH on test wikis"" .. Revert "Revert "Se

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Revert "Revert "Set wgCommentTableSchemaMigrationStage = MIG...

2017-12-04 Thread Krinkle (Code Review)
Hello Urbanecm, Legoktm, jenkins-bot, Anomie, Zoranzoki21, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/395149 to review the following change. Change subject: Revert "Revert "Set wgCommentTableSchemaMigrationStage = MIGRATION_WRITE_BOTH on test wikis"" ..

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: multiversion: Update docs for 'wfShellWikiCmd' hook

2017-12-04 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395120 ) Change subject: multiversion: Update docs for 'wfShellWikiCmd' hook .. multiversion: Update docs for 'wfShellWikiCmd' hook This hook was al

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Remove various redundant '@license' tags in file headers

2017-12-04 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395074 ) Change subject: Remove various redundant '@license' tags in file headers .. Remove various redundant '@license' tags in file headers Redund

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: [DNM] Fun with PHP - https://3v4l.org/Gv3QB

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394732 ) Change subject: [DNM] Fun with PHP - https://3v4l.org/Gv3QB .. [DNM] Fun with PHP - https://3v4l.org/Gv3QB Change-Id: I40bae7c35c8225c84739

[MediaWiki-commits] [Gerrit] labs...guc[master]: Add tests for App::getDB()

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394705 ) Change subject: Add tests for App::getDB() .. Add tests for App::getDB() Change-Id: I49bd2762b070d8641d8a7c7faf139a823d2d0421 --- M src/App

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: build: Add PHP 7.1 to Travis CI test matrix

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394666 ) Change subject: build: Add PHP 7.1 to Travis CI test matrix .. build: Add PHP 7.1 to Travis CI test matrix Looks like we've got at least on

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: objectcache: Fix HashBagOStuffTest test in PHP 7.1

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394665 ) Change subject: objectcache: Fix HashBagOStuffTest test in PHP 7.1 .. objectcache: Fix HashBagOStuffTest test in PHP 7.1 > There was 1 erro

[MediaWiki-commits] [Gerrit] labs...guc[master]: build: Update to PHPUnit 5

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394661 ) Change subject: build: Update to PHPUnit 5 .. build: Update to PHPUnit 5 Bug: T181852 Change-Id: I5134c4fe4c3398e5e0463472601e813ed975cded

[MediaWiki-commits] [Gerrit] integration/config[master]: Switch tools/guc jobs from php55 to php56

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394655 ) Change subject: Switch tools/guc jobs from php55 to php56 .. Switch tools/guc jobs from php55 to php56 Looks like a base image variant exis

[MediaWiki-commits] [Gerrit] labs...guc[master]: Switch to `*.web.db.svc.eqiad.wmflabs`

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394651 ) Change subject: Switch to `*.web.db.svc.eqiad.wmflabs` .. Switch to `*.web.db.svc.eqiad.wmflabs` Bug: T176686 Change-Id: I2631422949cd8af2c

[MediaWiki-commits] [Gerrit] labs...guc[master]: Fix-up typo from 9f2c515

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394645 ) Change subject: Fix-up typo from 9f2c515 .. Fix-up typo from 9f2c515 Change-Id: I6a14d52bee225a80b4f6afc3b1947baba09771de --- M src/App.php

[MediaWiki-commits] [Gerrit] labs...guc[master]: Improve re-use of connections between getDB()/openDB()

2017-12-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394644 ) Change subject: Improve re-use of connections between getDB()/openDB() .. Improve re-use of connections between getDB()/openDB() Current de

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Update "Other patches" to hide if already reviewed by MW or ...

2017-11-30 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394511 ) Change subject: Update "Other patches" to hide if already reviewed by MW or WMF groups .. Update "Other patches" to hide if already reviewe

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Update "Other patches" to hide if already reviewed by MW or ...

2017-11-30 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394511 ) Change subject: Update "Other patches" to hide if already reviewed by MW or WMF groups .. Update "Other patches" to hide if already

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: objectcache: Add @ingroup to RedisBagOStuff

2017-11-30 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394505 ) Change subject: objectcache: Add @ingroup to RedisBagOStuff .. objectcache: Add @ingroup to RedisBagOStuff Was missing from "Cache" group,

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Remove @defgroup from DefaultSettings.php

2017-11-30 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394502 ) Change subject: Remove @defgroup from DefaultSettings.php .. Remove @defgroup from DefaultSettings.php This renders in Doxygen as a blank p

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: resourceloader: Remove deprecated minifier config vars

2017-11-30 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394497 ) Change subject: resourceloader: Remove deprecated minifier config vars .. resourceloader: Remove deprecated minifier config vars Unused, th

[MediaWiki-commits] [Gerrit] mediawiki...NavigationTiming[master]: Factor out getRandom() and inSample()

2017-11-30 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394441 ) Change subject: Factor out getRandom() and inSample() .. Factor out getRandom() and inSample() * Centralise the Math.floor() logic and add

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: jobs: Remove ClearUserWatchlistJob 'batchSize' option

2017-11-30 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394400 ) Change subject: jobs: Remove ClearUserWatchlistJob 'batchSize' option .. jobs: Remove ClearUserWatchlistJob 'batchSize' option Use $wgUpdat

[MediaWiki-commits] [Gerrit] mediawiki/core[REL1_30]: Shell: skip null parameters

2017-11-29 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394184 ) Change subject: Shell: skip null parameters .. Shell: skip null parameters Right now they're treated as empty strings, however this doesn't

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: [WIP] WebStart: Remove use of realpath() for $IP

2017-11-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/393713 ) Change subject: [WIP] WebStart: Remove use of realpath() for $IP .. [WIP] WebStart: Remove use of realpath() for $IP When installing MediaW

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Exclude CR<0 if from performance group

2017-11-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/393702 ) Change subject: Exclude CR<0 if from performance group .. Exclude CR<0 if from performance group We already exclude CR -2 which is limited

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Exclude CR<0 if from performance group

2017-11-27 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393702 ) Change subject: Exclude CR<0 if from performance group .. Exclude CR<0 if from performance group We already exclude CR -2 which is

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Also exclude WIP/DNM from "Other" section

2017-11-27 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393701 ) Change subject: Also exclude WIP/DNM from "Other" section .. Also exclude WIP/DNM from "Other" section Matches the logic of the oth

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Also exclude WIP/DNM from "Other" section

2017-11-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/393701 ) Change subject: Also exclude WIP/DNM from "Other" section .. Also exclude WIP/DNM from "Other" section Matches the logic of the other secti

[MediaWiki-commits] [Gerrit] operations/puppet[production]: Gerrit: Fix negative cut-off logo on page load

2017-11-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/393691 ) Change subject: Gerrit: Fix negative cut-off logo on page load .. Gerrit: Fix negative cut-off logo on page load Follows-up db9ea750ef. Sim

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Update "Other patches" to use reviewerin: instead of file:

2017-11-27 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393641 ) Change subject: Update "Other patches" to use reviewerin: instead of file: .. Update "Other patches" to use reviewerin: instead of f

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Update "Other patches" to use reviewerin: instead of file:

2017-11-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/393641 ) Change subject: Update "Other patches" to use reviewerin: instead of file: .. Update "Other patches" to use reviewerin: instead of file: We

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Add section for patches by others to subscribed areas

2017-11-27 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393638 ) Change subject: Add section for patches by others to subscribed areas .. Add section for patches by others to subscribed areas We s

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Add section for patches by others to subscribed areas

2017-11-27 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/393638 ) Change subject: Add section for patches by others to subscribed areas .. Add section for patches by others to subscribed areas We should ke

[MediaWiki-commits] [Gerrit] mediawiki...EventLogging[master]: RemoteSchema: Use SchemaApiUrl instead of DBname in cache key

2017-11-22 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392986 ) Change subject: RemoteSchema: Use SchemaApiUrl instead of DBname in cache key .. RemoteSchema: Use SchemaApiUrl instead of DBname in cache k

[MediaWiki-commits] [Gerrit] mediawiki...EventLogging[master]: Simplify onSetup check for configuration

2017-11-22 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392982 ) Change subject: Simplify onSetup check for configuration .. Simplify onSetup check for configuration * No need for isset(), because these a

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: InstallDocFormatter: Add missing @covers scope

2017-11-22 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392891 ) Change subject: InstallDocFormatter: Add missing @covers scope .. InstallDocFormatter: Add missing @covers scope The format() method is a o

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: build: Add 'npm run qunit' command

2017-11-22 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392888 ) Change subject: build: Add 'npm run qunit' command .. build: Add 'npm run qunit' command The running of 'grunt qunit' is unconvenient due t

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Revert "Move styles for Special:UserRights to separate style...

2017-11-21 Thread Krinkle (Code Review)
Hello Bartosz DziewoƄski, Fomafix, Reedy, Florianschmidtwelzow, jenkins-bot, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/392773 to review the following change. Change subject: Revert "Move styles for Special:UserRights to separate style module" .

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: multiversion: Assume --wiki=aawiki for purgeUrls.php

2017-11-20 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392561 ) Change subject: multiversion: Assume --wiki=aawiki for purgeUrls.php .. multiversion: Assume --wiki=aawiki for purgeUrls.php Similar to pur

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: noc: Remove more unused styles/images and update Vector

2017-11-20 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392557 ) Change subject: noc: Remove more unused styles/images and update Vector .. noc: Remove more unused styles/images and update Vector No visib

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: noc: Link to Grafana instead of Ganglia

2017-11-19 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392317 ) Change subject: noc: Link to Grafana instead of Ganglia .. noc: Link to Grafana instead of Ganglia Closes https://github.com/wikimedia/oper

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Try again with using 'ownerin:' instead of hardcoded ...

2017-11-17 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/392112 ) Change subject: Try again with using 'ownerin:' instead of hardcoded list .. Try again with using 'ownerin:' instead of hardcoded li

[MediaWiki-commits] [Gerrit] performance[refs/meta/dashboards/custom]: Try again with using 'ownerin:' instead of hardcoded ...

2017-11-17 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392112 ) Change subject: Try again with using 'ownerin:' instead of hardcoded list .. Try again with using 'ownerin:' instead of hardcoded list Foll

[MediaWiki-commits] [Gerrit] mediawiki...TimedMediaHandler[master]: mw.MediaWikiPlayerSupport: Fix more .bind() deprecation warn...

2017-11-16 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/391991 ) Change subject: mw.MediaWikiPlayerSupport: Fix more .bind() deprecation warnings .. mw.MediaWikiPlayerSupport: Fix more .bind() deprecation

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: labs: Enable profiler based on same signals as production does

2017-11-16 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/391987 ) Change subject: labs: Enable profiler based on same signals as production does .. labs: Enable profiler based on same signals as production

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: labs: Clean up outdated wgProfiler config

2017-11-16 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/391982 ) Change subject: labs: Clean up outdated wgProfiler config .. labs: Clean up outdated wgProfiler config * Define $wgProfiler cleanly (not au

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: wikidatawiki back to wmf.7

2017-11-15 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/391755 ) Change subject: wikidatawiki back to wmf.7 .. wikidatawiki back to wmf.7 Follows-up 75a290113. Pushing local unstaged change from tin.eqia

[MediaWiki-commits] [Gerrit] mediawiki/extensions[master]: Archive the ActivityMonitor extension

2017-11-15 Thread Krinkle (Code Review)
Krinkle has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/391644 ) Change subject: Archive the ActivityMonitor extension .. Archive the ActivityMonitor extension Bug: T180631 Change-Id: I92b097d1220

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: [WIP] Split profile.php from StartProfiler and add PhpAutoPr...

2017-11-13 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/391162 ) Change subject: [WIP] Split profile.php from StartProfiler and add PhpAutoPrepend .. [WIP] Split profile.php from StartProfiler and add Php

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: VectorTemplate: Restore 'id' for siteNotice wrapper

2017-11-13 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/391158 ) Change subject: VectorTemplate: Restore 'id' for siteNotice wrapper .. VectorTemplate: Restore 'id' for siteNotice wrapper Follows-up bc47b

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Improve StartProfiler.php file documentation

2017-11-10 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390548 ) Change subject: Improve StartProfiler.php file documentation .. Improve StartProfiler.php file documentation In preparation for refactoring

[MediaWiki-commits] [Gerrit] labs...guc[master]: frontend: Restore replag warning

2017-11-10 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390543 ) Change subject: frontend: Restore replag warning .. frontend: Restore replag warning It was still trying to insert before `.maincontent for

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Add warning and documentation comment to HHVMRequestInit.php

2017-11-10 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390526 ) Change subject: Add warning and documentation comment to HHVMRequestInit.php .. Add warning and documentation comment to HHVMRequestInit.php

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Setup: Include StartProfiler before others

2017-11-09 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390351 ) Change subject: Setup: Include StartProfiler before others .. Setup: Include StartProfiler before others Bug: T180183 Change-Id: Ibcf78d094

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Merge ProfilerFunctions into GlobalFunctions

2017-11-09 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390350 ) Change subject: Merge ProfilerFunctions into GlobalFunctions .. Merge ProfilerFunctions into GlobalFunctions Even if people use these (depr

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: VectorTemplate: Some consistency in mixed PHP/HTML formatting

2017-11-08 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390186 ) Change subject: VectorTemplate: Some consistency in mixed PHP/HTML formatting .. VectorTemplate: Some consistency in mixed PHP/HTML formatti

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: VectorTemplate: Refactor literal HTML by using Html methods ...

2017-11-08 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390187 ) Change subject: VectorTemplate: Refactor literal HTML by using Html methods instead .. VectorTemplate: Refactor literal HTML by using Html

[MediaWiki-commits] [Gerrit] operations/puppet[production]: webperf: Refactor tests to directly associate expected data ...

2017-11-08 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390083 ) Change subject: webperf: Refactor tests to directly associate expected data with cases .. webperf: Refactor tests to directly associate exp

[MediaWiki-commits] [Gerrit] operations/puppet[production]: webperf: Record navtiming discards to Graphite, and add is_s...

2017-11-08 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/390061 ) Change subject: webperf: Record navtiming discards to Graphite, and add is_sane test .. webperf: Record navtiming discards to Graphite, and

[MediaWiki-commits] [Gerrit] mediawiki...CentralAuth[wmf/1.31.0-wmf.7]: Use the proper cache key method in loadFromCache()

2017-11-07 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/389917 ) Change subject: Use the proper cache key method in loadFromCache() .. Use the proper cache key method in loadFromCache() Bug: T17 Chang

[MediaWiki-commits] [Gerrit] mediawiki...EventLogging[master]: hooks: Remove redundant wgMemc check in onSetup

2017-11-07 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/389887 ) Change subject: hooks: Remove redundant wgMemc check in onSetup .. hooks: Remove redundant wgMemc check in onSetup This is run on all reque

[MediaWiki-commits] [Gerrit] mediawiki...Flow[master]: Reduce initFlowExtension cost from Utils::isParsoidConfigured()

2017-11-07 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/389886 ) Change subject: Reduce initFlowExtension cost from Utils::isParsoidConfigured() .. Reduce initFlowExtension cost from Utils::isParsoidConfig

[MediaWiki-commits] [Gerrit] operations/puppet[production]: grafana: Fix network graphs to not end in a weird drop to 0

2017-11-07 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/389798 ) Change subject: grafana: Fix network graphs to not end in a weird drop to 0 .. grafana: Fix network graphs to not end in a weird drop to 0

[MediaWiki-commits] [Gerrit] mediawiki...EventLogging[master]: build: Remove outdated references to server/

2017-11-07 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/389795 ) Change subject: build: Remove outdated references to server/ .. build: Remove outdated references to server/ The development version of the

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: upload: Simplify UploadStashTest by using getNewTempFile()

2017-11-06 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/389664 ) Change subject: upload: Simplify UploadStashTest by using getNewTempFile() .. upload: Simplify UploadStashTest by using getNewTempFile() Th

[MediaWiki-commits] [Gerrit] mediawiki...WikimediaEvents[master]: Add modules/ sub directories to match module registration

2017-11-03 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/388573 ) Change subject: Add modules/ sub directories to match module registration .. Add modules/ sub directories to match module registration * On

[MediaWiki-commits] [Gerrit] mediawiki...WikimediaEvents[master]: geoFeatures: Use local var instead of document data for state

2017-11-03 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/388571 ) Change subject: geoFeatures: Use local var instead of document data for state .. geoFeatures: Use local var instead of document data for sta

[MediaWiki-commits] [Gerrit] operations/puppet[production]: grafana: Update server-board JSON to schemaVersion 14 (no-op)

2017-11-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/387980 ) Change subject: grafana: Update server-board JSON to schemaVersion 14 (no-op) .. grafana: Update server-board JSON to schemaVersion 14 (no-o

[MediaWiki-commits] [Gerrit] operations/puppet[production]: grafana: Reduce "network" dropdown to valid options for curr...

2017-11-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/387981 ) Change subject: grafana: Reduce "network" dropdown to valid options for current server .. grafana: Reduce "network" dropdown to valid optio

[MediaWiki-commits] [Gerrit] mediawiki/core[REL1_29]: Fix case of SpecialRecentChanges class

2017-11-01 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/387973 ) Change subject: Fix case of SpecialRecentChanges class .. Fix case of SpecialRecentChanges class Test fails with $wgAutoloadAttemptLowercas

  1   2   3   4   5   6   7   8   9   10   >