[18:28:29] RainbowSprinkles: When will RELEASE-NOTES-1.30 get created in master? [18:28:44] Today :) [18:28:50] I'm working on branching 29 now [18:29:50] * anomie went to add release notes to a new patch, but the 1.30 file wasn't there yet [18:30:40] Branch script running now [18:31:20] core, vendor, skins all branched, working on extensions [18:31:28] +2'd the master bump [18:40:49] anomie: Master is 1.30 now [18:40:59] thanks [18:41:06] Well, another 30 seconds [18:41:07] grrrr [18:41:52] Ugh, composer.lock broken in rel1_29 [18:41:54] Weird [19:56:32] Woo branch time :) [20:06:35] Composer is busted on rel1_29 though, jenkins ain't happy [20:08:58] how is it busted? [20:09:49] 19:44:43 oojs/oojs-ui: 0.21.2 installed, 0.21.1 required. [20:09:49] 19:44:43 Error: your composer.lock file is not up to date. Run "composer update" to install newer dependencies [20:10:15] https://gerrit.wikimedia.org/r/#/c/350661/ will fix it [20:10:32] gotcha [20:11:37] yay to CI catching real problems :) [20:11:44] Indeed! [20:29:09] anomie: in E:ORES, where are the ApiHooks::onAPIGetAllowedParams and co getting registered? [20:30:28] heh, they look a bit orphaned [20:31:52] so it's not just me, good [20:40:29] legoktm: Hmm, it looks like I7472c9fc91700048999342605979d6446a552efc removed the hooks for some reason. [20:41:29] That would probably explain why I didn't see the parameters the other day when I was looking at the API help... [20:42:02] oh :/ [20:43:12] It looks like T163687 is the task to turn it back on. tgr might know more. [20:43:12] T163687: Re-enable ORES data in action API - https://phabricator.wikimedia.org/T163687 [20:43:24] ok [20:43:38] mostly I was trying to look at an example of how to add parameters to core modules and got confused [20:44:17] What sort of parameter are you looking at adding? [20:45:01] I wanted to add a siprop=linterstats option to meta=siteinfo [20:46:24] As a new prop, you'd probably want to add a new hook in the 'default' switch case in ApiQuerySiteinfo::execute() [20:46:45] OTOH, it might be better to just make a new meta module instead of adding more junk to meta=siteinfo. [20:48:09] * anomie notes that if the whole dependency injection thing is ever completely done, ApiQuerySiteinfo would probably have 50 things being injected [20:49:29] I was looking at APIAfterExecute but really didn't get that far. a new meta module seems way easier [23:32:26] I spent half an hour yesterday trying to figure out how to get console debug logging in eval.php so that I could debug etcd in deployment-prep, didn't get there [23:32:41] I think maybe it should just be a separate SPI