check-commit: check capitalization in summary lines
authorEric Sumner <ericsumner@fb.com>
Thu, 05 Feb 2015 14:09:08 -0800
changeset 24049 ba272156113f
parent 24048 13d88b7eb3a0
child 24050 a9b61dbdb827
check-commit: check capitalization in summary lines At the moment, check-commit will complain about the topic being capitalized, but not the summary that comes after it. This diff corrects that deficiency.
contrib/check-commit
--- a/contrib/check-commit	Thu Feb 05 10:57:45 2015 -0800
+++ b/contrib/check-commit	Thu Feb 05 14:09:08 2015 -0800
@@ -25,6 +25,7 @@
     (r"^# .*\n(?!merge with )[^#]\S+[^:] ",
      "summary line doesn't start with 'topic: '"),
     (r"^# .*\n[A-Z][a-z]\S+", "don't capitalize summary lines"),
+    (r"^# .*\n[^\n]*: *[A-Z][a-z]\S+", "don't capitalize summary lines"),
     (r"^# .*\n.*\.\s+$", "don't add trailing period on summary line"),
     (r"^# .*\n.{78,}", "summary line too long"),
     (r"^\+\n \n", "adds double empty line"),