- User Since
- Jan 31 2016, 7:15 AM (185 w, 2 d)
- Remove clean rule
- Use more readable names
- Address Pavel's comments.
Thanks, Ismail. LGTM!
- Feedback Greg.
- Remove dlang types.
Thanks, Jan. This LGTM with Pavel's comments addressed.
Sun, Aug 18
This code review seems to address a few things. I think it might have been easier to discuss if they were separate patches, but here are my comments:
Fri, Aug 16
Chris is correct, PYTHON_EXECUTABLE will be an absolute path. For this to work you'll need to remove /usr/bin/env.
Make RegularExpression POSIX compliant.
LGTM with the inline comment.
Thu, Aug 15
Wed, Aug 14
This test seems to be failing pretty much everywhere. Can you please have a look or should we revert it?