[arvados-dev] updated: baa79212b746fa0df0f360d1c7984876e6a2d53f

git repository hosting git at public.arvados.org
Thu Jun 1 13:33:26 UTC 2023


Summary of changes:
 git/hooks/coding-standards.rb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

       via  baa79212b746fa0df0f360d1c7984876e6a2d53f (commit)
      from  8be7dc90e3f6f4d6cfb058bcbeb79fa09608ab32 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit baa79212b746fa0df0f360d1c7984876e6a2d53f
Author: Tom Clegg <tom at curii.com>
Date:   Thu Jun 1 09:31:53 2023 -0400

    Accept "resolves #" (equivalent to "closes #"), update help message.
    
    no issue #
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/git/hooks/coding-standards.rb b/git/hooks/coding-standards.rb
index 15a975c..bc4d649 100755
--- a/git/hooks/coding-standards.rb
+++ b/git/hooks/coding-standards.rb
@@ -62,7 +62,7 @@ $broken_commit_message = /Please enter a commit message to explain why this merg
 $wrong_way_merge_main = /Merge( remote-tracking)? branch '([^\/]+\/)?main' into/
 $merge_main = /Merge branch '[^']+'((?! into)| into main)/
 $pull_merge = /Merge branch 'main' of /
-$refs_or_closes_or_no_issue = /(refs #|closes #|fixes #|no issue #)/i
+$refs_or_closes_or_no_issue = /(refs #|closes #|fixes #|resolves #|no issue #)/i
 
 # enforced custom commit message format
 def check_message_format
@@ -132,8 +132,8 @@ def check_message_format
     # this test will complain about *every* commit in the merge otherwise, obscuring
     # the real reason for the rejection (the no_ff merge)
     if not no_ff and not $refs_or_closes_or_no_issue.match(message)
-      puts "\n[POLICY] All commits to main must include an issue using \"refs #\" or"
-      puts "\"closes #\", or specify \"no issue #\"\n"
+      puts "\n[POLICY] All commits to main must include an issue using \"refs #\","
+      puts "\"fixes #\", \"closes #\", or \"resolves #\", or specify \"no issue #\"\n"
       puts "\n******************************************************************\n"
       puts "\nOffending commit: #{rev}\n"
       puts "\nOffending commit message:\n\n"

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list