aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBobosila Victor <vbobosila1@gmail.com>2021-09-25 12:52:50 +0300
committerBobosila Victor <vbobosila1@gmail.com>2021-09-25 12:52:50 +0300
commitad6106747761c597cb39b741a322dd608384b2ca (patch)
treed138a1516fe8e5cb1cd6644f2bc712ced3fd8fe0
parent57fc0c7d1ee907db44f91af123a47a308e6c3073 (diff)
downloadpuppet-hosts_core-ad6106747761c597cb39b741a322dd608384b2ca.tar.gz
puppet-hosts_core-ad6106747761c597cb39b741a322dd608384b2ca.tar.bz2
(MODULES-11197) Move commits rake task to rakelib/commits.rake
-rw-r--r--Rakefile25
-rw-r--r--rakelib/commits.rake25
2 files changed, 25 insertions, 25 deletions
diff --git a/Rakefile b/Rakefile
index 7f91a3f..e1d5f7b 100644
--- a/Rakefile
+++ b/Rakefile
@@ -84,28 +84,3 @@ EOM
end
end
-desc "verify that commit messages match CONTRIBUTING.md requirements"
-task(:commits) do
- # This rake task looks at the summary from every commit from this branch not
- # in the branch targeted for a PR.
- commit_range = 'HEAD^..HEAD'
- puts "Checking commits #{commit_range}"
- %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 /^\((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" \
- "\t\t(docs)\n" \
- "\t\t(docs)(DOCUMENT-<digits>)\n" \
- "\t\t(packaging)\n"
- "\t\t(maint)\n" \
- "\n\tThis test for the commit summary is case-insensitive.\n\n\n"
- else
- puts "#{commit_summary}"
- end
- puts "...passed"
- end
-end
diff --git a/rakelib/commits.rake b/rakelib/commits.rake
new file mode 100644
index 0000000..a900ee2
--- /dev/null
+++ b/rakelib/commits.rake
@@ -0,0 +1,25 @@
+desc "verify that commit messages match CONTRIBUTING.md requirements"
+task(:commits) do
+ # This rake task looks at the summary from every commit from this branch not
+ # in the branch targeted for a PR.
+ commit_range = 'HEAD^..HEAD'
+ puts "Checking commits #{commit_range}"
+ %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 /^\((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" \
+ "\t\t(docs)\n" \
+ "\t\t(docs)(DOCUMENT-<digits>)\n" \
+ "\t\t(packaging)\n"
+ "\t\t(maint)\n" \
+ "\n\tThis test for the commit summary is case-insensitive.\n\n\n"
+ else
+ puts "#{commit_summary}"
+ end
+ puts "...passed"
+ end
+end