diff options
author | Gheorghe Popescu <gheorghe.popescu@puppet.com> | 2020-10-06 14:08:52 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-10-06 14:08:52 +0300 |
commit | 834768ee9d048f6bcf1e000af6797a22eeb2cb8c (patch) | |
tree | 00620a4cb238266bddc512dbc809d32c900f82e8 /Rakefile | |
parent | 076ae4e58a5cc2a4aaed769c778f7a3cea54bd21 (diff) | |
parent | 5b07881851d60813d59a0b1c1bf2644d0901e769 (diff) | |
download | puppet-cron_core-834768ee9d048f6bcf1e000af6797a22eeb2cb8c.tar.gz puppet-cron_core-834768ee9d048f6bcf1e000af6797a22eeb2cb8c.tar.bz2 |
Merge pull request #37 from luchihoratiu/maint-github-actions-fixes
Diffstat (limited to 'Rakefile')
-rw-r--r-- | Rakefile | 25 |
1 files changed, 25 insertions, 0 deletions
@@ -84,3 +84,28 @@ 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 |