Code review comment for ~jslarraz/review-tools:use-logging-module

Revision history for this message
Alex Murray (alexmurray) wrote :

Thanks for the very extensive comments to help walk-through this MR - it was very valuable. I really like this cleanup and it makes the code more pythonic by using the standard logging module, plus I actually like the CLI printing the errors directly rather than including them in the json output. LGTM!

review: Approve

« Back to merge proposal