[00:02:34] ori-l: ping [00:46:44] New patchset: Jeroen De Dauw; "Update URL to point to GitHub rather then MediaWiki.org" [mediawiki/extensions/ParserHooks] (master) - https://gerrit.wikimedia.org/r/73602 [02:59:46] New review: Parent5446; "Two minor things and we're pretty much good to go." [mediawiki/extensions/Annotator] (master) - https://gerrit.wikimedia.org/r/71364 [03:03:05] Change merged: Yaron Koren; [mediawiki/extensions/SemanticForms] (master) - https://gerrit.wikimedia.org/r/73571 [03:39:55] New patchset: Brian Wolff; "Make AssembleUploadChunksJob give a status update when it starts." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73605 [04:03:44] New patchset: Brian Wolff; "Make AssembleUploadChunksJob give a status update when it starts." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73605 [04:34:30] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/72324 [04:51:15] New patchset: Rasel160; "WIP, do not merge. Will allow dynamic form generation in the details step." [mediawiki/extensions/UploadWizard] (master) - https://gerrit.wikimedia.org/r/66561 [04:52:33] New review: Dan-nl; "hey brian wolff," [mediawiki/core] (master) C: 1; - https://gerrit.wikimedia.org/r/67229 [05:00:25] legoktm: pong [05:00:41] ohai [05:01:02] so on https://gerrit.wikimedia.org/r/#/c/69303/, i did all the things you said to do, but what did you mean by "Explain what you are doing in your commit message"? [05:02:32] it's laconic and the pronouns are ambiguous [05:03:23] 'By default this is disabled' -> what's "this", and do you mean it is disabled in production now and you're enabling it, or that you're adding a feature that is off by default? [05:03:40] 'It is only viewable' -> what is "it"? [05:03:50] hmm ok. [05:04:12] I can improve on that [05:04:28] gtg afk now, I'll post a new patchset later tonight. thanks! [05:05:05] and although you link to the bug, you could explain the rationale (it'd provide for a way to prune the blacklist) [05:05:08] np [05:14:33] New patchset: Rasel160; "WIP, do not merge. Will allow dynamic form generation in the details step." [mediawiki/extensions/UploadWizard] (master) - https://gerrit.wikimedia.org/r/66561 [05:19:09] New review: Rasel160; "(1 comment)" [mediawiki/extensions/UploadWizard] (master) - https://gerrit.wikimedia.org/r/66561 [08:33:59] New patchset: Ori.livneh; "Add 'env' module for managing shell environments" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/73606 [08:46:46] New patchset: Ori.livneh; "Add 'env' module for managing shell environments" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/73606 [08:48:22] Change merged: jenkins-bot; [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/73606 [09:34:23] New patchset: TTO; "(bug 51206) add namespace names in Chechen language" [mediawiki/extensions/Scribunto] (master) - https://gerrit.wikimedia.org/r/73607 [09:47:52] New review: Zaran; "Please read the comments in EditProofreadPagePage" [mediawiki/extensions/ProofreadPage] (pagePagesRefactoring) C: -1; - https://gerrit.wikimedia.org/r/72933 [10:23:31] New patchset: Malvineous; "Update for MW 1.21" [mediawiki/extensions/MassEditRegex] (master) - https://gerrit.wikimedia.org/r/73609 [10:23:31] New patchset: Malvineous; "Update messages now functionality has changed" [mediawiki/extensions/MassEditRegex] (master) - https://gerrit.wikimedia.org/r/73610 [10:25:29] New patchset: Nemo bis; "Move translators credits list to a wiki page" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73611 [10:26:22] New review: Nemo bis; "Hashar, it's at https://gerrit.wikimedia.org/r/#/c/73611/ now" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/68381 [10:34:51] New review: Malvineous; "Localisation text changes only" [mediawiki/extensions/MassEditRegex] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/73610 [10:36:41] New review: Malvineous; "Approving my own commit as extension maintainer, lacking any other reviewers on this project." [mediawiki/extensions/MassEditRegex] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/73609 [10:36:42] Change merged: Malvineous; [mediawiki/extensions/MassEditRegex] (master) - https://gerrit.wikimedia.org/r/73609 [10:37:32] Change merged: Malvineous; [mediawiki/extensions/MassEditRegex] (master) - https://gerrit.wikimedia.org/r/73610 [10:56:24] how difficult would it be to implement some kind of blame functionality? [10:56:41] I would assume the db is not optimal for such lookup [11:05:59] New patchset: Rjain; "Added the create class" [mediawiki/extensions/Annotator] (master) - https://gerrit.wikimedia.org/r/71364 [11:24:51] New patchset: Rjain; "Separated the OKFN modules to mediawiki.utils.okfn/" [mediawiki/extensions/Annotator] (master) - https://gerrit.wikimedia.org/r/73612 [11:29:55] New patchset: Rjain; "Separated the OKFN modules to mediawiki.utils.okfn/" [mediawiki/extensions/Annotator] (master) - https://gerrit.wikimedia.org/r/73612 [11:57:46] New review: Nikerabbit; "Looks harmless, but rather than empty string which is meaningless, you could use something like 'en'..." [mediawiki/extensions/UniversalLanguageSelector] (master) C: -1; - https://gerrit.wikimedia.org/r/73448 [12:18:59] New patchset: Amire80; "WIP: Trying to write a test for bug 50790" [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/72365 [12:25:12] New patchset: Nikerabbit; "Update CHANGELOG and README" [mediawiki/extensions/Translate] (master) - https://gerrit.wikimedia.org/r/73613 [13:16:39] New patchset: Amire80; "WIP: Trying to write a test for bug 50790" [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/72365 [13:18:29] New patchset: Amire80; "WIP: Trying to write a test for bug 50790" [qa/browsertests] (master) - https://gerrit.wikimedia.org/r/72365 [14:26:59] New patchset: Esanders; "Add more specific class to cite errors" [mediawiki/extensions/Cite] (master) - https://gerrit.wikimedia.org/r/73096 [15:13:42] New patchset: Anomie; "Link, not display, files on [[Special:Notifications]]" [mediawiki/extensions/Thanks] (master) - https://gerrit.wikimedia.org/r/73615 [15:29:14] Change merged: jenkins-bot; [mediawiki/extensions/LiquidThreads] (master) - https://gerrit.wikimedia.org/r/73167 [15:30:01] Change merged: jenkins-bot; [mediawiki/extensions/WikiEditor] (master) - https://gerrit.wikimedia.org/r/73165 [16:01:48] New patchset: Hoo man; "Don't cache the gadget-defintion in case no gadgets were found" [mediawiki/extensions/Gadgets] (master) - https://gerrit.wikimedia.org/r/73617 [16:02:02] hi devunt_ [16:02:12] hi [16:02:18] how's it going? [16:02:31] hi sucheta [16:03:00] Hey sumanah! [16:04:10] great [16:04:16] :p [16:04:23] devunt: I'm sorry, I do not understand [16:05:26] sumanah, So, I had sort of a birthday yesterday. Wish me a belated one ^_^ [16:05:42] http://notes.wikimediadc.org/p/english-lessons - so, in the next couple of hours, an English tutor is going to swing by there and help people out with some editing, if they'd like some help improving their English writing skills [16:07:39] so, for instance, if you have written an email to wikitech-l or a blog post or something, and you would like an English tutor to check that it's in correct English and give you tips to improve your writing for next time, you can put it on http://notes.wikimediadc.org/p/english-lessons and Erika will help you [16:31:28] New review: Reedy; "(1 comment)" [mediawiki/extensions/Gadgets] (master) - https://gerrit.wikimedia.org/r/73617 [16:40:37] New review: Raimond Spekking; "(1 comment)" [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/73415 [16:44:38] Change merged: jenkins-bot; [mediawiki/extensions/Gadgets] (master) - https://gerrit.wikimedia.org/r/73617 [16:45:17] New patchset: Reedy; "Update Gadgets to master" [mediawiki/core] (wmf/1.22wmf9) - https://gerrit.wikimedia.org/r/73618 [16:45:46] New patchset: Reedy; "Update Gadgets to master" [mediawiki/core] (wmf/1.22wmf10) - https://gerrit.wikimedia.org/r/73619 [16:51:25] Change merged: jenkins-bot; [mediawiki/core] (wmf/1.22wmf9) - https://gerrit.wikimedia.org/r/73618 [16:52:51] Change merged: jenkins-bot; [mediawiki/core] (wmf/1.22wmf10) - https://gerrit.wikimedia.org/r/73619 [16:57:00] Reedy: Shall we close the bug? [16:57:26] Probably not [16:58:24] New patchset: MaxSem; "bump cache version" [mediawiki/extensions/Gadgets] (master) - https://gerrit.wikimedia.org/r/73620 [16:59:19] MaxSem: Is this related to my change? [17:00:08] He's invalidating ALL OF THE CACHES [17:00:09] oh, I see.. :P [17:01:05] don't sync it yet [17:26:36] New patchset: Hoo man; "Parse the undelete-search-prefix message" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73415 [17:27:09] New review: Hoo man; "(1 comment)" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73415 [17:27:24] New review: Hoo man; "Addressed Raimond's comment" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73415 [17:30:20] New review: Nikerabbit; "Please do not set Verified+2 yourself, that's for Jenkins." [mediawiki/extensions/SemanticForms] (master) - https://gerrit.wikimedia.org/r/73571 [17:52:30] New patchset: Jeroen De Dauw; "Add packagist badges to README" [mediawiki/extensions/ParserHooks] (master) - https://gerrit.wikimedia.org/r/73623 [17:52:48] New patchset: Jeroen De Dauw; "Add packagist badges to README" [mediawiki/extensions/ParserHooks] (master) - https://gerrit.wikimedia.org/r/73623 [17:52:53] Change merged: Jeroen De Dauw; [mediawiki/extensions/ParserHooks] (master) - https://gerrit.wikimedia.org/r/73623 [17:56:01] New patchset: Jeroen De Dauw; "Add packagist badges to README" [mediawiki/extensions/Serialization] (master) - https://gerrit.wikimedia.org/r/73624 [17:56:11] Change merged: Jeroen De Dauw; [mediawiki/extensions/Serialization] (master) - https://gerrit.wikimedia.org/r/73624 [18:00:37] New patchset: Physikerwelt; "skip tex filter by default" [mediawiki/extensions/Math] (LaTeXML) - https://gerrit.wikimedia.org/r/73625 [18:00:37] New patchset: Physikerwelt; "Adding some special test functions" [mediawiki/extensions/Math] (LaTeXML) - https://gerrit.wikimedia.org/r/73626 [18:02:09] Change merged: jenkins-bot; [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73415 [18:02:46] New patchset: Physikerwelt; "skip tex filter by default" [mediawiki/extensions/Math] (LaTeXML) - https://gerrit.wikimedia.org/r/73625 [18:03:02] New patchset: Physikerwelt; "skip tex filter by default" [mediawiki/extensions/Math] (LaTeXML) - https://gerrit.wikimedia.org/r/73625 [18:03:45] New patchset: Physikerwelt; "add some special case test functions" [mediawiki/extensions/Math] (LaTeXML) - https://gerrit.wikimedia.org/r/73625 [18:04:05] Change merged: Physikerwelt; [mediawiki/extensions/Math] (LaTeXML) - https://gerrit.wikimedia.org/r/73625 [18:07:27] New patchset: Physikerwelt; "add newline after error message" [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73628 [18:08:03] Change merged: Physikerwelt; [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73628 [18:09:05] should tests/phpunit/phpunit.php be able to pass --testsuite extensions? [18:21:19] New review: Yaron Koren; "Oops - sorry on both counts. I actually didn't know about not setting "Verified +2" - I've set that ..." [mediawiki/extensions/SemanticForms] (master) - https://gerrit.wikimedia.org/r/73571 [18:31:47] Change merged: Jeroen De Dauw; [mediawiki/extensions/SemanticMediaWiki] (master) - https://gerrit.wikimedia.org/r/73570 [18:31:56] New review: Jeroen De Dauw; "Thanks kgh!" [mediawiki/extensions/SemanticMediaWiki] (master) - https://gerrit.wikimedia.org/r/73570 [18:58:27] New patchset: Physikerwelt; "Update FormulaInfo to the new Math extension" [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73632 [19:01:49] New review: Owora; "Great!" [mediawiki/extensions/BiblioPlus] (master) C: 1; - https://gerrit.wikimedia.org/r/72685 [19:03:33] New patchset: Physikerwelt; "Update depricated function call to new function" [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73633 [19:08:41] Change merged: Physikerwelt; [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73632 [19:11:07] New patchset: Physikerwelt; "Change base class of MathObject from MathRenderer to MathLaTeXML" [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73634 [19:12:01] Change merged: Physikerwelt; [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73633 [19:12:08] Change merged: Physikerwelt; [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73634 [19:20:09] New patchset: KarenEddy; "Critical Update: Fixes an infinite loop in an xml parsing function." [mediawiki/extensions/BiblioPlus] (master) - https://gerrit.wikimedia.org/r/72685 [19:21:30] New review: Aaron Schulz; "Now is a good time to just changed the whole status stuff to use memcached instead. I originally had..." [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/73605 [20:06:04] New patchset: Se4598; "Add triggered filter id as parameter to messages" [mediawiki/extensions/AbuseFilter] (master) - https://gerrit.wikimedia.org/r/71291 [20:20:37] New review: Mollywhite; "Going to go ahead and merge this in so the CSS can be used with the new navbar." [mediawiki/extensions/BookManagerv2] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/72742 [20:24:24] New patchset: Physikerwelt; "Bugfix in mathpagestat table" [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73709 [20:25:27] New patchset: Physikerwelt; "Bugfix in mathpagestat table" [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73709 [20:25:56] Change merged: Physikerwelt; [mediawiki/extensions/MathSearch] (master) - https://gerrit.wikimedia.org/r/73709 [20:31:41] New patchset: Rtdwivedi; "Addition of code that will be used for setting up of editing system in PHP instead of JS." [mediawiki/extensions/ProofreadPage] (pagePagesRefactoring) - https://gerrit.wikimedia.org/r/72933 [20:41:57] New patchset: Esanders; "Allow postEdit confirmation to be triggered asynchronously" [mediawiki/core] (master) - https://gerrit.wikimedia.org/r/69855 [21:02:58] anyone has vim config for mw code style standard? [21:06:28] AzaToth: there's something on [[CC]] iirc [21:51:24] New patchset: Siebrand; "Remove title case" [mediawiki/extensions/OAuth] (master) - https://gerrit.wikimedia.org/r/73712 [21:53:16] New review: MZMcBride; "(1 comment)" [mediawiki/extensions/OAuth] (master) - https://gerrit.wikimedia.org/r/73712 [21:57:58] New review: Siebrand; "(1 comment)" [mediawiki/extensions/OAuth] (master) - https://gerrit.wikimedia.org/r/73389 [21:58:20] New patchset: Siebrand; "Add explicit GENDER for log entries" [mediawiki/extensions/OAuth] (master) - https://gerrit.wikimedia.org/r/73713 [22:09:02] New patchset: Ori.livneh; "Run MediaWiki unit tests via 'vagrant run-tests'" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/73714 [22:36:01] New patchset: Ori.livneh; "Run MediaWiki unit tests via 'vagrant run-tests'" [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/73714 [22:36:47] Change merged: jenkins-bot; [mediawiki/vagrant] (master) - https://gerrit.wikimedia.org/r/73714 [22:53:22] New review: CSteipp; "(1 comment)" [mediawiki/extensions/CentralAuth] (master) - https://gerrit.wikimedia.org/r/73549 [23:25:41] New review: Krinkle; "Overriding the message is imho an odd use case, but if we're going to support it we should probably ..." [mediawiki/core] (master) C: -1; - https://gerrit.wikimedia.org/r/69855 [23:49:39] New patchset: Siebrand; "Add namespace names in Chechen language (ce)" [mediawiki/extensions/Scribunto] (master) - https://gerrit.wikimedia.org/r/73607 [23:49:48] New review: Siebrand; "Thanks." [mediawiki/extensions/Scribunto] (master) C: 2; - https://gerrit.wikimedia.org/r/73607 [23:51:46] Change merged: jenkins-bot; [mediawiki/extensions/Scribunto] (master) - https://gerrit.wikimedia.org/r/73607 [23:56:57] New patchset: Siebrand; "Add code for setting up of editing system in PHP instead of JS" [mediawiki/extensions/ProofreadPage] (pagePagesRefactoring) - https://gerrit.wikimedia.org/r/72933 [23:57:46] New review: Siebrand; "(1 comment)" [mediawiki/extensions/ProofreadPage] (pagePagesRefactoring) C: -1; - https://gerrit.wikimedia.org/r/72933