From 3e7ea029311fb5b7682761748efe06d771a5dbbf Mon Sep 17 00:00:00 2001 From: Michael Hashizume Date: Fri, 1 Dec 2023 11:25:19 -0800 Subject: (PA-5395) Use reusable static code analysis This commit updates the static code analysis GitHub Actions workflow from using a copy in this repository to a resuable copy in the Phoenix team centralized GitHub Actions repository. In addition, this removes the commits Rake task as it is no longer used in the reusable version of this workflow. --- rakelib/commits.rake | 25 ------------------------- 1 file changed, 25 deletions(-) delete mode 100644 rakelib/commits.rake (limited to 'rakelib') diff --git a/rakelib/commits.rake b/rakelib/commits.rake deleted file mode 100644 index 7cee9fa..0000000 --- a/rakelib/commits.rake +++ /dev/null @@ -1,25 +0,0 @@ -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. - 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 /^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-)\n" \ - "\t\t(MODULES-)\n" \ - "\t\t(docs)\n" \ - "\t\t(docs)(DOCUMENT-)\n" \ - "\t\t(packaging)\n" - "\t\t(maint)\n" \ - "\t\tRelease prep v\n" \ - "\n\tThis test for the commit summary is case-insensitive.\n\n\n" - else - puts "#{commit_summary}" - end - puts "...passed" - end -end -- cgit v1.2.3