python - コミット前にソースコードをチェックする pre-commit フック

Question

コミット前にソースコードをチェックする簡単なスクリプトを検索します。

このパターンを使用したいので、「logging.info(...)」を含むすべてのファイルを拒否します。

import logging
logger=logging.getLogger(__name__)
del(logging)

#...
logger.info()

Answer 1

解決策が見つからなかったので、自分で書きました：

#!/usr/bin/env python
# scp svn@svnserver:REPOSITORY/hooks/
# ssh svn@svnserver chmod a+rx REPOSITORY/hooks/pre-commit
# Above chmod is important!
import re, sys, subprocess
sys.stdout=sys.stderr
svnlookPath = 'svnlook'

not_allowed=[
    (re.compile(r'^\s*logging\.(error|info|debug).*'), u'use logger!'),
    ]

def main():
    changed=[]

    # sys.argv ['/svn/modwork/hooks/pre-commit', '/svn/modwork', '4508-bm']
    pipe = subprocess.Popen([svnlookPath, 'changed', sys.argv[1], '--transaction', sys.argv[2]], stdout=subprocess.PIPE)
    for line in pipe.stdout:
        change_type, fn = line.split(' ', 1)
        if not change_type[0] in 'AUC':
            continue # skip, not a source code change: delete or meta data update.
        changed.append(fn.strip())
    ret=pipe.wait()
    if ret:
        sys.exit(ret)
    for fn in changed:
        if not fn.endswith('.py'):
            continue
        pipe = subprocess.Popen([svnlookPath, 'cat', sys.argv[1], '--transaction', sys.argv[2], fn], stdout=subprocess.PIPE)
        for line in pipe.stdout:
            for regex, help in not_allowed:
                if regex.match(line):
                    print 'Content in file %s is not allowed: %s: %r' % (fn, help, line.strip())
                    sys.exit(1)
                #print 'OK', fn, repr(line)

        ret=pipe.wait()
        if ret:
            sys.exit(ret)

if __name__=='__main__':
    main()