- User Since
- Jun 18 2016, 2:24 PM (260 w, 5 d)
Thanks for looking into this! Looks like a great improvement!
Looks good, thanks!
Thanks a lot for the work! This looks fine to land and build on.
LGTM (but I think you already landed this)
Can you fix the description?
We should update the docs as well.
Missing the test?
Tue, Jun 15
LG thanks! This has been on the TODO for a while, but just never found the time.
Thanks, took another pass. Probably just one more after this one.
Thanks! Starting working through, mostly stylistic nits while digesting the design.
Mon, Jun 14
Nice, thanks for the fix!
Sat, Jun 12
Fri, Jun 11
Thu, Jun 10
Thanks, starting to work through it.
Thanks. (Sorry, missed this somehow)
Wed, Jun 9
Tue, Jun 8
This looks right to me, but might be good to get another LGTM.
I thought you abandoned this change? Looks like you landed it.
Mon, Jun 7
Thanks for the review, PTAL.
Are the test failures related? They look like they might be.
Can you add a link to the discussion for this?
Fri, Jun 4
Thu, Jun 3