[00:03:25] (03CR) 10Jforrester: [C: 04-1] "Should also alter eventLogger.html" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/221315 (owner: 10Divec) [00:12:34] (03PS2) 10Divec: Fix imetests to have compliant code style [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/221315 [00:13:05] (03CR) 10Jforrester: "recheck" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/221315 (owner: 10Divec) [00:14:23] (03CR) 10Divec: [C: 032] "Great news" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/221162 (owner: 10Jforrester) [00:15:48] (03PS3) 10Jforrester: IME tests: Make compliant with our code style [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/221315 (owner: 10Divec) [00:15:54] (03CR) 10Jforrester: [C: 032] IME tests: Make compliant with our code style [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/221315 (owner: 10Divec) [00:16:39] (03Merged) 10jenkins-bot: Run the faked IME unit tests in VE-MW too [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/221162 (owner: 10Jforrester) [00:18:13] (03Merged) 10jenkins-bot: IME tests: Make compliant with our code style [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/221315 (owner: 10Divec) [00:20:45] (03PS3) 10Jforrester: IME tests: Add OS X native Japanese Hiragana & Katakana [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/221278 [00:21:52] (03PS1) 10Divec: Update generated data for Unicode 8.0.0 [unicodejs] - 10https://gerrit.wikimedia.org/r/221321 [00:24:47] (03CR) 10Jforrester: [C: 032] Update generated data for Unicode 8.0.0 [unicodejs] - 10https://gerrit.wikimedia.org/r/221321 (owner: 10Divec) [00:25:29] (03Merged) 10jenkins-bot: Update generated data for Unicode 8.0.0 [unicodejs] - 10https://gerrit.wikimedia.org/r/221321 (owner: 10Divec) [00:39:30] (03PS1) 10Divec: Strong directionality support [unicodejs] - 10https://gerrit.wikimedia.org/r/221323 [00:42:21] (03PS2) 10Divec: Strong directionality support [unicodejs] - 10https://gerrit.wikimedia.org/r/221323 [00:43:26] (03CR) 10Jforrester: [C: 031] Strong directionality support [unicodejs] - 10https://gerrit.wikimedia.org/r/221323 (owner: 10Divec) [00:58:37] (03PS1) 10Jforrester: package.json: Fix licence per new npm style [unicodejs] - 10https://gerrit.wikimedia.org/r/221325 [00:58:39] (03PS1) 10Jforrester: build: Bump development dependencies [unicodejs] - 10https://gerrit.wikimedia.org/r/221326 [00:58:41] (03PS1) 10Jforrester: build: Add phantomjs dependency to suppress warning [unicodejs] - 10https://gerrit.wikimedia.org/r/221327 [00:59:17] (03PS2) 10Jforrester: build: Bump development dependencies [unicodejs] - 10https://gerrit.wikimedia.org/r/221326 [01:00:30] (03PS2) 10Jforrester: build: Add phantomjs dependency to suppress warning [unicodejs] - 10https://gerrit.wikimedia.org/r/221327 [01:00:56] (03CR) 10Legoktm: [C: 032] package.json: Fix licence per new npm style [unicodejs] - 10https://gerrit.wikimedia.org/r/221325 (owner: 10Jforrester) [01:01:04] legoktm: Thanks. [01:01:23] (03CR) 10Legoktm: [C: 032] build: Bump development dependencies [unicodejs] - 10https://gerrit.wikimedia.org/r/221326 (owner: 10Jforrester) [01:01:53] (03Merged) 10jenkins-bot: package.json: Fix licence per new npm style [unicodejs] - 10https://gerrit.wikimedia.org/r/221325 (owner: 10Jforrester) [01:02:08] (03Merged) 10jenkins-bot: build: Bump development dependencies [unicodejs] - 10https://gerrit.wikimedia.org/r/221326 (owner: 10Jforrester) [01:02:11] (03CR) 10Legoktm: [C: 032] build: Add phantomjs dependency to suppress warning [unicodejs] - 10https://gerrit.wikimedia.org/r/221327 (owner: 10Jforrester) [01:03:18] (03Merged) 10jenkins-bot: build: Add phantomjs dependency to suppress warning [unicodejs] - 10https://gerrit.wikimedia.org/r/221327 (owner: 10Jforrester) [01:38:29] 10VisualEditor, 7Easy: extensions/VisualEditor - file permissions trigger debian lintian warnings - https://phabricator.wikimedia.org/T104044#1406544 (10wpmirrordev) 3NEW a:3Krenair [01:42:24] (03PS1) 10Jforrester: Set file permissions correctly for *.php files [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/221332 (https://phabricator.wikimedia.org/T104044) [16:09:57] (03CR) 10Mvolz: "Just added two more tests, for jagged arrays in TD (which are allowed by templateData) and for jagged arrays in citation data (unlikely to" [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/218946 (https://phabricator.wikimedia.org/T97044) (owner: 10Mvolz) [16:26:50] 10Citoid: Enable Citoid in nowiki - https://phabricator.wikimedia.org/T103878#1407239 (10Mvolz) 1. All available types and fields are indeed at https://aurimasv.github.io/z2csl/typeMap.xml One non-obvious thing about fields there is that the specific type is used where a base type is available (i.e. websiteTit... [16:42:00] 10Citoid: Enable Citoid in nowiki - https://phabricator.wikimedia.org/T103878#1407253 (10Mvolz) Here's a better explanation of fields: https://www.mediawiki.org/wiki/Citoid/Maps_TemplateData#Available_fields_and_field_structure [16:55:18] Looks like VE still adds erroneous tags. [17:02:06] Have you reported it? Or is it already known? [17:03:31] It's actually one of the oldest issues with VE. I thought it had been fixed but I just ran into it again. [17:03:54] There was even an abusefilter on enwiki to tag such edits. [17:05:09] It'd be caused by parts of VE which I am unfamiliar with. [17:05:29] Not clear to me whether we're discussing one issue or a group of different issues that show up in the same way to the user. [17:06:11] https://en.wikipedia.org/wiki/Special:AbuseFilter/550 [17:07:52] So... You're looking at any nowiki addition? [17:09:16] Looks like there hasn't been any hit by a VE edit recently. [17:09:36] You realise that nowiki tags are an actual feature of MediaWiki, not just a placeholder used by editors when things go wrong, right? [17:10:27] Yes. :) There was a time when almost all hits by that filter was by VE edits. [17:10:31] (added erroneously) [17:12:44] Okay, so what edits do you see there from VE which you think are errors? [17:13:58] As I already said, I don't see any recent hits on that filter but I just ran into it while trying to edit. [17:14:06] saw it when viewing diff [17:14:37] with a
as well [17:17:02] Well, if you can't find an existing bug for it you could file one. [17:17:32] Some existing resolved tasks I've found suggest that these can potentially be caused by Parsoid. [17:19:49] (03PS9) 10Mvolz: Allow structured citoid fields to be flattened [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/218946 (https://phabricator.wikimedia.org/T97044) [17:20:14] https://phabricator.wikimedia.org/T55548 [17:20:19] is one of the open ones [17:20:53] (03PS10) 10Mvolz: Allow structured citoid fields to be flattened [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/218946 (https://phabricator.wikimedia.org/T97044) [17:20:55] Not having your edit's diff, I have no idea if that's the problem or not [17:24:13] Also hi Glaisher, I didn't recognise your nick [17:25:57] Hi. [17:26:12] FWIW, I was trying to reply to a discussion when this happened. [17:26:14] :P [17:26:20] (03CR) 10Alex Monk: [C: 032] Set file permissions correctly for *.php files [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/221332 (https://phabricator.wikimedia.org/T104044) (owner: 10Jforrester) [17:26:29] so maybe it might be declined :P [17:26:50] 10VisualEditor, 7Easy, 5Patch-For-Review: extensions/VisualEditor - file permissions trigger debian lintian warnings - https://phabricator.wikimedia.org/T104044#1407290 (10Krenair) 5Open>3Resolved a:5Krenair>3Jdforrester-WMF [17:27:49] Philon, I can't guarantee things won't be declined, but if your issue is reproducible in a reasonable way outside of discussion pages, I don't see why it would be declined over that. [17:28:26] (03Merged) 10jenkins-bot: Set file permissions correctly for *.php files [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/221332 (https://phabricator.wikimedia.org/T104044) (owner: 10Jforrester) [17:32:54] Hm, got added because I ~~~~'d it. And the blockquote got added because I indented it. Not a valid bug after all. [17:33:02] Why is not doing a : type of indent instead? [19:59:56] 10VisualEditor, 7Easy, 5Patch-For-Review, 5WMF-deploy-2015-06-30_(1.26wmf12): extensions/VisualEditor - file permissions trigger debian lintian warnings - https://phabricator.wikimedia.org/T104044#1407541 (10wpmirrordev) I confirm that the above mentioned debian lintian warning is cured. Thanks.