[00:09:09] niedzielski: is lastSuccessfulBuild/artifact from the last build or last merged build? I would like to make sure that we only consider merged code for the Alpha builds. [00:09:40] bearND: i was thinking i could make a new job that is only used for merges [00:09:56] bearND: not our verification job [00:10:00] niedzielski: cool. That would be good. [00:26:22] bmansurov: that happened so quick :( [00:26:30] sorry [13:47:21] hullo [13:47:42] * phuedx has just got back from the doctor [14:12:31] it's very hard to concentrate when you have water trickling out of your ears [14:13:51] joakino: just emailed you about strategy dates. would you please reply-all on that? figured I better hit you up on IRC early (for me) today so you see it. I want to get stuff in process with travel today, even if that might mean some remote participation. [14:14:28] joakino: I'm going for a jog shortly and then getting ready for work and so on. ttyl [15:44:45] it's quiet in here these days [15:46:37] hey phuedx [15:46:50] why do you have water trickling out your ears? [15:48:37] i got my ears syringed [15:48:49] sounds fun? [15:48:56] because i had water trapped behind my eardrum but i had too much earwax to let it out :/ [15:49:08] it's a very awkward feeling [15:49:15] but now i hear in super high definition [15:49:18] FlorianSW: i have to confess i didn't test the original patch. i was hoping the reviewer would do that for me :P [15:49:36] phuedx: like when you go to the dentist and your teeth feel strangely clean only... ears! [15:49:42] and I think, that bmansurov thought, that you tested it : [15:49:43] :D [15:49:49] jdlrobson: exactly [15:49:53] woah [15:49:58] jdlrobson: quick question about https://gerrit.wikimedia.org/r/#/c/231188/1 [15:50:03] i wanna do this and then go to the dentists [15:50:08] and have __both__ [15:50:17] maybe let myself go a little blind too so i need new glasses [15:50:23] phuedx: sure [15:50:26] that would be greaty [15:50:30] since the resource loader queue is flushed when the mediawiki.startUp module is loaded (at the bottom, right?) [15:50:36] then why are we using events at all [15:50:51] or is this change supposed to be the start of a series of changes cleaning that little thing up? [15:51:02] a start of a series of events [15:51:07] okie poke [15:51:14] so the immediate thing to do is not use the if statement since the js is no longer blocking [15:51:19] and it's useless right now [15:51:24] i'll ask for clarification in the commit message [15:51:28] using this makes us get that working again [15:51:38] but yes now all the js is synced it might not make as much sense.. but... [15:51:44] and there's a comment that needs updating in [15:51:45] oh yeah [15:51:50] get it working first -- obvs [15:51:55] mobile.head still gets top loaded from what i understand [15:52:00] it just doesn't block [15:52:15] so it might finish downloading and start executing prior to the rest of the page loading [15:52:25] it's asyncified? [15:52:33] ^ should be the name of the attribute [15:52:33] yup [15:52:45] if that happens, the event will fire before the bottom scripts load unless i'm missing something? [15:52:54]