[01:19:12] (03PS3) 10Krinkle: Properly return API failure to MassMessageJob [extensions/MassMessage] - 10https://gerrit.wikimedia.org/r/576140 (https://phabricator.wikimedia.org/T246217) (owner: 10Ppchelko) [01:22:20] (03CR) 10Krinkle: "According to mw:Maintainers, Editing-team is steward which I'm not sure is correct, but maybe? Lead maintainer is Legoktm." [extensions/MassMessage] - 10https://gerrit.wikimedia.org/r/576140 (https://phabricator.wikimedia.org/T246217) (owner: 10Ppchelko) [02:08:44] (03CR) 10Nray: "@Phuedx and @VolkerE: Thank you both for your thoughtful reviews! Both of them have made me rethink a couple things in my approach. I hope" [skins/Vector] - 10https://gerrit.wikimedia.org/r/579293 (https://phabricator.wikimedia.org/T243281) (owner: 10Nray) [03:06:27] 10Africa-Wikimedia-Developers, 10Outreach-Programs-Projects, 10Technical-Tool-Request, 10Google-Summer-of-Code (2020): Design and Develop a tool to correct false depicts claims manually on Wikimedia Commons - https://phabricator.wikimedia.org/T245758 (10Urstrulyvishu) Hi everyone, Myself Vishwas Singh, an... [05:29:11] 10Pywikibot, 10Pywikibot-Documentation: Update manuals - remove sysopnames references - https://phabricator.wikimedia.org/T242552 (10Xqt) [05:29:17] 10Pywikibot, 10Pywikibot-login.py, 10Patch-For-Review: dualism between user and sysop needs to be overtaken - https://phabricator.wikimedia.org/T71283 (10Xqt) [05:33:33] 10Pywikibot, 10Pywikibot-login.py: -user: is ignored for sysop actions - https://phabricator.wikimedia.org/T98607 (10Xqt) 05Open→03Declined Dualism has been dropped [05:56:43] 10Pywikibot, 10Pywikibot-login.py: dualism between user and sysop needs to be overtaken - https://phabricator.wikimedia.org/T71283 (10Xqt) [06:46:02] 10Pywikibot, 10Patch-Needs-Improvement, 10Pywikibot-Wikidata: Harvest_template should be able to add reciprocal claims - https://phabricator.wikimedia.org/T173238 (10Xqt) [06:48:19] 10Pywikibot, 10Pywikibot-Scripts, 10Patch-Needs-Improvement, 10Performance Issue, and 2 others: reflinks should fetch only 1Mb of each linked document - https://phabricator.wikimedia.org/T124138 (10Xqt) [06:49:25] 10Pywikibot, 10Patch-Needs-Improvement: selflink.py - AttributeError: 'SelflinkBot' object has no attribute 'pageToUnlink' - https://phabricator.wikimedia.org/T150694 (10Xqt) [08:09:38] (03PS1) 10Thiemo Kreuz (WMDE): Fix a ton of easy to fix PHPCS issues [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/580180 [11:05:37] (03CR) 10D3r1ck01: [C: 03+1] "Looks good to me! Let another eye have a look. Thanks @Thiemo :)" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/580180 (owner: 10Thiemo Kreuz (WMDE)) [13:37:24] (03CR) 10Polishdeveloper: [C: 04-1] "I like this code but personally I'm against this solution. My concern is that developer will have to decide whether his feature depends on" (032 comments) [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) (owner: 10Phuedx) [13:40:09] (03PS2) 10Yaron Koren: Fix a ton of easy to fix PHPCS issues [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/580180 (owner: 10Thiemo Kreuz (WMDE)) [14:33:52] (03CR) 10Yaron Koren: [C: 03+2] Fix a ton of easy to fix PHPCS issues [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/580180 (owner: 10Thiemo Kreuz (WMDE)) [14:33:58] (03CR) 10Yaron Koren: [C: 03+2] "Thank you!" [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/580180 (owner: 10Thiemo Kreuz (WMDE)) [14:37:14] (03Merged) 10jenkins-bot: Fix a ton of easy to fix PHPCS issues [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/580180 (owner: 10Thiemo Kreuz (WMDE)) [18:12:47] (03PS2) 10Phuedx: featureManager: Add DynamicConfigRequirement requirement [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) [18:15:06] (03CR) 10Phuedx: featureManager: Add DynamicConfigRequirement requirement (032 comments) [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) (owner: 10Phuedx) [18:16:01] (03CR) 10Phuedx: featureManager: Add DynamicConfigRequirement requirement (031 comment) [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) (owner: 10Phuedx) [18:24:59] (03CR) 10jerkins-bot: [V: 04-1] featureManager: Add DynamicConfigRequirement requirement [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) (owner: 10Phuedx) [19:32:39] (03PS3) 10Phuedx: featureManager: Add DynamicConfigRequirement requirement [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) [19:45:45] (03CR) 10jerkins-bot: [V: 04-1] featureManager: Add DynamicConfigRequirement requirement [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) (owner: 10Phuedx) [20:58:35] (03PS1) 10Jdlrobson: Clearly distinguish local variables from globals [skins/Vector] - 10https://gerrit.wikimedia.org/r/580467 [21:11:35] (03CR) 10jerkins-bot: [V: 04-1] Clearly distinguish local variables from globals [skins/Vector] - 10https://gerrit.wikimedia.org/r/580467 (owner: 10Jdlrobson) [21:27:08] (03PS2) 10Jdlrobson: Clearly distinguish local variables from globals [skins/Vector] - 10https://gerrit.wikimedia.org/r/580467 [21:39:37] (03CR) 10jerkins-bot: [V: 04-1] Clearly distinguish local variables from globals [skins/Vector] - 10https://gerrit.wikimedia.org/r/580467 (owner: 10Jdlrobson) [22:04:50] (03CR) 10Polishdeveloper: "just a minor" (031 comment) [skins/Vector] - 10https://gerrit.wikimedia.org/r/579619 (https://phabricator.wikimedia.org/T244481) (owner: 10Phuedx) [22:24:19] (03Abandoned) 10Jdlrobson: Don't use variables for values only ever used once [skins/Vector] - 10https://gerrit.wikimedia.org/r/579603 (https://phabricator.wikimedia.org/T247537) (owner: 10Jdlrobson) [22:24:26] (03Abandoned) 10Jdlrobson: Clearly distinguish local variables from globals [skins/Vector] - 10https://gerrit.wikimedia.org/r/580467 (owner: 10Jdlrobson)