[00:32:05] jdlrobson: Hi, currently I'm working on your task (https://phabricator.wikimedia.org/T170579 ), btw do you know how to run the unittests (just to make sure everything are all right when I do the change) on MobileFrontend extension ? [01:09:17] jdlrobson: and for `MediaWiki.Files.OneClassPerFile.MultipleFound` rule, all of the violations are in the Test module, should I fix it ? [01:10:30] ^most of them are class that used for mocking the test [01:11:04] ^and that also affects `MediaWiki.Files.ClassMatchesFilename.NotMatch` [12:25:13] jdlrobson: any ide to make this code: https://dpaste.de/FmT1 [12:25:14] MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment.NewLineComment compliant ? [12:25:46] *idea [12:32:51] euhm, this https://dpaste.de/QwE5 as well :/ [18:14:28] bearND: what do you think about continuing to merge media and references stuff before the next summary switchover attempt? should we wait, or just go ahead and get stuff merged? [18:14:40] i'm inclined to be cautious myself [18:14:46] cc coreyfloyd [18:15:06] mdholloway: I'm leaning towards postponing those merges until next year, tbh. [18:15:27] bearND: same here [20:58:20] tgr: o/ Has something changed in the expected format of the 'project' parameter for reading list entries? [20:58:29] I'm getting "'en.wikipedia.org' is not a recognized project." [20:59:43] yeah, it should be https://en.wikipedia.org [21:00:03] which is how SiteMatrix returns it [21:00:33] cool, thanks! [21:00:35] I can change it back if it turns out to be problematic, it was just less WMF-specific this way [21:00:47] nope, that's easily done [21:09:50] tgr: I'm also getting a HTTP 405 error when hitting the PUT /data/lists/{id} endpoint [21:15:49] mdholloway: do you want to prep another MCS deploy? (But wait for the all clear from the Services guys before actually deploying it anywhere) [21:15:59] bearND: sure [21:16:10] mdholloway: thanks [21:25:08] dbrant: hm, looks like that's a varnish config issue [21:25:14] will check with the ops [21:42:50] bearND: rolled back because of a bug in the section checking. should i revert that and redeploy so we can get the announcments update in? [21:43:19] mdholloway: do you have a stack trace or something? [21:43:33] https://www.irccloud.com/pastebin/tRaWHkdS/ [21:44:23] mdholloway: ugh, ok. Revert that for now. But we still get the dc:modified fix in, right? [21:44:46] bearND: yes, we'll get that in [21:47:21] bearND: on the bright side, we have a few examples of main namespace pages that show the behavior we're checking for now. [21:47:51] ru.wikipedia.org/v1/page/mobile-sections/Ki-61 [21:48:11] ar.wikipedia.org/v1/page/mobile-sections/ويكيبيديا:الميدان_(تصفح) [21:48:22] oops, looks like that one isn't main namespace [21:49:04] I get 404s [21:49:06] it is still mostly in other namespaces: User, Talk, Template, Draft [21:49:39] bearND: https://ru.wikipedia.org/api/rest_v1/page/html/Ki-61 [21:49:53] ah, right. ty [21:53:00] coreyfloyd mdholloway: In light of the issues we've had the past two days, I think we should postpone the summary switch to early next year. Thoughts? [21:53:35] bearND: thats fine… the only thing we have to do is the announcements update [21:53:47] coreyfloyd: i'm about to deploy that. [21:53:56] 👍 [21:54:35] coreyfloyd: ok, was going to check with you first before I tell the Services team [21:54:36] yeah, postponing the switchover seems like the wisest way to go. [22:00:57] coreyfloyd: Is there a good task to keep other in the loop about this postponement? [22:01:02] others [22:08:22] bearND: coreyfloyd: T179875? [22:08:23] T179875: Update RESTBase to get summary content from MCS Summary 2.0 endpoint when development is complete - https://phabricator.wikimedia.org/T179875 [22:09:21] bearND: let me see… sorry in a meeting [22:09:33] mdholloway: Good. Looks like mobrovac added the comment already. Thanks! :) [22:09:43] coreyfloyd: no worries, got it figured [22:11:23] bearND: cool