Code review comment for ~jslarraz/review-tools:add-pre-commit-hook

Revision history for this message
Jorge Sancho Larraz (jslarraz) wrote (last edit ):

Added a couple of commits on top

https://git.launchpad.net/~jslarraz/review-tools/commit/?id=dd72f359ce84c75e6502e829da1a31f65ac1c9af run black on some files that were not included or explicitly excluded in run-helper. After inspecting the output of black I don't see a good reason to don't format them properly. It will be great if you can take a look to double check I'm not missing anything there.

https://git.launchpad.net/~jslarraz/review-tools/commit/?id=52a3f0fbb8e2db4cc4926a416e18a2850dcf4942 moves test runners to makefile and update pre-commit hook accordingly. **Some special considerations as inline comments**

« Back to merge proposal