aboutsummaryrefslogtreecommitdiff
path: root/rakelib
diff options
context:
space:
mode:
authorMichael Hashizume <michael.hashizume@puppet.com>2023-09-14 15:46:28 -0700
committerMichael Hashizume <michael.hashizume@puppet.com>2023-09-14 15:49:37 -0700
commit6205984f35d02698eb07d3061412651bf0912ca4 (patch)
treed4026283b6e87095c96dbe2f900cfa8cc9e75c81 /rakelib
parent4ff5b442e1abc9cb201e8e4a8fbc50d59114d19d (diff)
downloadpuppet-augeas_core-6205984f35d02698eb07d3061412651bf0912ca4.tar.gz
puppet-augeas_core-6205984f35d02698eb07d3061412651bf0912ca4.tar.bz2
(maint) Update commit check
This commit updates the commits Rake check to include commit summaries that begin with the Puppet Agent (PA) Jira project and removes references to a non-existent CONTRIBUTION.md.
Diffstat (limited to 'rakelib')
-rw-r--r--rakelib/commits.rake11
1 files changed, 5 insertions, 6 deletions
diff --git a/rakelib/commits.rake b/rakelib/commits.rake
index 42eb209..7cee9fa 100644
--- a/rakelib/commits.rake
+++ b/rakelib/commits.rake
@@ -1,4 +1,4 @@
-desc "verify that commit messages match CONTRIBUTING.md requirements"
+desc "verify that commit summaries are properly formatted"
task(:commits) do
# This rake task looks at the summary from every commit from this branch not
# in the branch targeted for a PR.
@@ -7,11 +7,10 @@ task(:commits) do
%x{git log --no-merges --pretty=%s #{commit_range}}.each_line do |commit_summary|
# This regex tests for the currently supported commit summary tokens.
# The exception tries to explain it in more full.
- if /^Release prep|\((maint|packaging|doc|docs|modules-\d+)\)|revert/i.match(commit_summary).nil?
- raise "\n\n\n\tThis commit summary didn't match CONTRIBUTING.md guidelines:\n" \
- "\n\t\t#{commit_summary}\n" \
- "\tThe commit summary (i.e. the first line of the commit message) should start with one of:\n" \
- "\t\t(MODULES-<digits>) # this is most common and should be a ticket at tickets.puppet.com\n" \
+ if /^Release prep|\((maint|packaging|doc|docs|modules|pa-\d+)\)|revert/i.match(commit_summary).nil?
+ raise "\n\n\n\tPlease make sure that your commit summary (i.e. the first line of the commit message) starts with one of the following:\n" \
+ "\t\t(PA-<digits>)\n" \
+ "\t\t(MODULES-<digits>)\n" \
"\t\t(docs)\n" \
"\t\t(docs)(DOCUMENT-<digits>)\n" \
"\t\t(packaging)\n"