[12:21:35] jdlrobson: here [12:22:24] jdlrobson: I'll remove my -1, I was just curious on your take as it looks messy to anybody with CSS exposure on a daily base [12:22:54] jdlrobson: as even with new rules there is no order at all and props are once here, once there [13:04:37] bmansurov: Hey Baha, just saw two errors in a patch +2ed by you [13:04:53] bmansurov: https://gerrit.wikimedia.org/r/#/c/366968/3 [13:05:19] bmansurov: don't wanna remove your +2 without telling first [13:05:39] Volker_E, o/, let me take a look [13:06:52] Volker_E, how come style lint errors aren't caught by the system? [13:07:32] I was reaching out to James F on -editing for the same question [13:07:44] something must have gone wrong in the config last couple of weeks [13:08:06] Volker_E, ok, I'll remove my +2 for now, thanks for the catch [13:08:28] bmansurov: alright :) [13:32:05] mdholloway: is build tools update ready to merge? [13:32:43] dbrant: i think so [13:32:53] unfortunately, i'm still not clear on what was causing trouble the other day [13:33:04] (in fact, i haven't a clue) [13:34:00] separately, though, last night i worked out a way to persist the sdk in an appropriate location on our CI instances rather than re-downloading with each run [13:34:38] oh that's great. I assume that will speed things up quite a bit? [13:34:43] just need to clean up/document and probably get a second opinion from the CI overlords [13:34:53] it knocks about a minute off the build. [13:34:59] excellent [18:00:19] mdholloway: is the compilation detail patch roughly ready? [18:00:32] yep, ready for review [18:00:40] so is the follow-up [18:03:53] mdholloway: the fake-out toolbar patch is approved, too [18:04:16] dbrant: excellent!