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

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

Thanks for this @jslarraz - I wonder if perhaps we should adapt the existing run-black etc scripts to be able to operate on a list of files, then we can make sure we run it with the same arguments etc as what is done in lpci?

Or if instead the hooks should just call `run-black` etc and run against all the files, and not worry about limiting it to the list of changed files...

Thoughts?

« Back to merge proposal