changeset 24049:ba272156113f

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.
author Eric Sumner <ericsumner@fb.com>
date Thu, 05 Feb 2015 14:09:08 -0800
parents 13d88b7eb3a0
children a9b61dbdb827
files contrib/check-commit
diffstat 1 files changed, 1 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- 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"),