[00:57:22] TimStarling: I'm updating parsertests.txt [01:12:05] so I should wait for PS 4? I'm ready to +2 it now [02:07:16] James_F: that's it, merge conflicts [03:07:22] MaxSem: Fun. :-) [03:51:36] AaronSchulz: I'm just going to merge it, you can put your tests in a separate commit [03:52:20] I wasted a lot of time with revisionstore DI and gave up on that approach [03:52:30] TimStarling: I want to amend the code though [03:52:42] it's got a +2 already [03:53:30] I held it back so I can refactor getCachedRevisionObject() a bit [04:23:25] TimStarling: OK, pushing a new version [11:40:34] duesen: re your recent DOD edits, it would be nice to include tarballed extensions in the definition of done (there's at least one not in production, ReplaceText) [13:27:19] tgr: you are right, feel free to change. doesn't make a difference in practice I think - the two extensions affected are Duplicator and DeletePagesForGood. I don't think either of them is bundled. [13:27:33] Where'S the list of bundled extensions anyway?... [15:13:05] duesen: https://phabricator.wikimedia.org/diffusion/MREL/browse/master/make-release/settings.yaml