Show More
@@ -1,56 +1,56 b'' | |||
|
1 | 1 | #!/usr/bin/env python |
|
2 | 2 | # |
|
3 | 3 | # Copyright 2014 Matt Mackall <mpm@selenic.com> |
|
4 | 4 | # |
|
5 | 5 | # A tool/hook to run basic sanity checks on commits/patches for |
|
6 | 6 | # submission to Mercurial. Install by adding the following to your |
|
7 | 7 | # .hg/hgrc: |
|
8 | 8 | # |
|
9 | 9 | # [hooks] |
|
10 | 10 | # pretxncommit = contrib/check-commit |
|
11 | 11 | # |
|
12 | 12 | # The hook can be temporarily bypassed with: |
|
13 | 13 | # |
|
14 | 14 | # $ BYPASS= hg commit |
|
15 | 15 | # |
|
16 | 16 | # See also: http://mercurial.selenic.com/wiki/ContributingChanges |
|
17 | 17 | |
|
18 | 18 | import re, sys, os |
|
19 | 19 | |
|
20 | 20 | errors = [ |
|
21 | 21 | (r"[(]bc[)]", "(BC) needs to be uppercase"), |
|
22 | 22 | (r"[(]issue \d\d\d", "no space allowed between issue and number"), |
|
23 | 23 | (r"[(]bug(\d|\s)", "use (issueDDDD) instead of bug"), |
|
24 | 24 | (r"^# User [^@\n]+$", "username is not an email address"), |
|
25 | 25 | (r"^# .*\n(?!merge with )[^#]\S+[^:] ", |
|
26 | 26 | "summary line doesn't start with 'topic: '"), |
|
27 | 27 | (r"^# .*\n[A-Z][a-z]\S+", "don't capitalize summary lines"), |
|
28 | 28 | (r"^# .*\n[^\n]*: *[A-Z][a-z]\S+", "don't capitalize summary lines"), |
|
29 | 29 | (r"^# .*\n.*\.\s+$", "don't add trailing period on summary line"), |
|
30 | 30 | (r"^# .*\n.{78,}", "summary line too long (limit is 78)"), |
|
31 | 31 | (r"^\+\n \n", "adds double empty line"), |
|
32 |
(r" |
|
|
32 | (r"^\+[ \t]+def [a-z]+_[a-z]", "adds a function with foo_bar naming"), | |
|
33 | 33 | ] |
|
34 | 34 | |
|
35 | 35 | node = os.environ.get("HG_NODE") |
|
36 | 36 | |
|
37 | 37 | if node: |
|
38 | 38 | commit = os.popen("hg export %s" % node).read() |
|
39 | 39 | else: |
|
40 | 40 | commit = sys.stdin.read() |
|
41 | 41 | |
|
42 | 42 | exitcode = 0 |
|
43 | 43 | for exp, msg in errors: |
|
44 | 44 | m = re.search(exp, commit, re.MULTILINE) |
|
45 | 45 | if m: |
|
46 | 46 | pos = 0 |
|
47 | 47 | for n, l in enumerate(commit.splitlines(True)): |
|
48 | 48 | pos += len(l) |
|
49 | 49 | if pos >= m.end(): |
|
50 | 50 | print "%d: %s" % (n, msg) |
|
51 | 51 | print " %s" % l[:-1] |
|
52 | 52 | if "BYPASS" not in os.environ: |
|
53 | 53 | exitcode = 1 |
|
54 | 54 | break |
|
55 | 55 | |
|
56 | 56 | sys.exit(exitcode) |
General Comments 0
You need to be logged in to leave comments.
Login now