Created
February 3, 2011 22:55
-
-
Save lentil/810399 to your computer and use it in GitHub Desktop.
PEP8 pre-commit hook in Python
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
#!/usr/bin/env python | |
from __future__ import with_statement | |
import os | |
import re | |
import shutil | |
import subprocess | |
import sys | |
import tempfile | |
def system(*args, **kwargs): | |
kwargs.setdefault('stdout', subprocess.PIPE) | |
proc = subprocess.Popen(args, **kwargs) | |
out, err = proc.communicate() | |
return out | |
def main(): | |
modified = re.compile('^[AM]+\s+(?P<name>.*\.py)', re.MULTILINE) | |
files = system('git', 'status', '--porcelain') | |
files = modified.findall(files) | |
tempdir = tempfile.mkdtemp() | |
for name in files: | |
filename = os.path.join(tempdir, name) | |
filepath = os.path.dirname(filename) | |
if not os.path.exists(filepath): | |
os.makedirs(filepath) | |
with file(filename, 'w') as f: | |
system('git', 'show', ':' + name, stdout=f) | |
output = system('pep8', '.', cwd=tempdir) | |
shutil.rmtree(tempdir) | |
if output: | |
print output, | |
sys.exit(1) | |
if __name__ == '__main__': | |
main() |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A custom script still has its uses (e.g. if you need to run other linting checks outside of the Python style checks), but nowadays I typically use the command line tool
flake8
(which incorporates thepep8
akapycodestyle
checks, and has a number of plugins available adding even more linting checks) with a configuration file as needed (e.g. setting the max-line-length), and its default git pre-commit hook, setup as follows:See http://flake8.pycqa.org/en/latest/user/using-hooks.html