Remove similar trailer keys from commit hook

lumynou5 · lumynou5 · commit b39491fa110a · 2025-03-04T13:39:03.000+08:00
The duplicated "Close" and "Closes" keys will possibly cause
inconsistency between commit messages.  The one being kept is "Closes"
because the implied subject is "this patch", so it should complete the
following sentence:

    This patch [closes]...

just like sentences in the body of a message.

"Bug" is also removed due to its similarity to "Closes a bug".

Change-Id: Idea7c0310249c603e550a7a4e27df490efe13d4c
diff --git a/scripts/commit-msg.hook b/scripts/commit-msg.hook
@@ -140,8 +140,7 @@ build_commit_trailer_regex() {
 
   # Predefined trailer keys.
   trailers=(
-    'CC' 'Change-Id'
-    'Bug' 'Close' 'Closes'
+    'CC' 'Change-Id' 'Closes'
     'Acked-by' 'Co-authored-by' 'Reported-by' 'Reviewed-by'
     'Signed-off-by' 'Suggested-by' 'Tested-by'
   )

Original file line number	Diff line number	Diff line change
`@@ -140,8 +140,7 @@ build_commit_trailer_regex() {`
`140`	`140`
`141`	`141`	`# Predefined trailer keys.`
`142`	`142`	`trailers=(`
`143`		`- 'CC' 'Change-Id'`
`144`		`- 'Bug' 'Close' 'Closes'`
	`143`	`+ 'CC' 'Change-Id' 'Closes'`
`145`	`144`	`'Acked-by' 'Co-authored-by' 'Reported-by' 'Reviewed-by'`
`146`	`145`	`'Signed-off-by' 'Suggested-by' 'Tested-by'`
`147`	`146`	`)`