Merge branch 'changelog-trailer-casing' into 'master'

Ensure trailers use the right casing

See merge request gitlab-org/gitaly!3558

1 files changed, 11 insertions, 1 deletions

diff --git a/danger/changelog/Dangerfile b/danger/changelog/Dangerfile
index 56efe44de..0aa9ff9ee 100644
--- a/danger/changelog/Dangerfile
+++ b/danger/changelog/Dangerfile
@@ -1,10 +1,20 @@
 require 'yaml'
 
 def lint_commit(commit)
-  trailer = commit.message.match(/^Changelog:\s*(?<category>\w+)/)
+  trailer = commit.message.match(/^(?<name>Changelog):\s*(?<category>.+)$/i)
 
   return :missing if trailer.nil? || trailer[:category].nil?
 
+  name = trailer[:name]
+
+  unless name == 'Changelog'
+    self.fail(
+      "The changelog trailer for commit #{commit.sha} must be `Changelog` (starting with a capital C), not `#{name}`"
+    )
+
+    return :invalid
+  end
+
   category = trailer[:category]
 
   return :valid if CATEGORIES.include?(category)