When a branch is being deleted, do not enforce the DCO signoff in all

the commits that are being deleted.

refs #11020

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <wvandewege@veritasgenetics.com>

diff --git a/git/hooks/enforce-dco-signoff.rb b/git/hooks/enforce-dco-signoff.rb
index c6245d275eba0feef387f43cf174224796c65918..ab93401d858ef564f436d4d377aa26a492d44fc7 100755 (executable)
--- a/git/hooks/enforce-dco-signoff.rb
+++ b/git/hooks/enforce-dco-signoff.rb
@@ -22,8 +22,8 @@ $arvados_DCO = /Arvados-DCO-1.1-Signed-off-by:/
 # enforced DCO signoff in commit message
 def check_message_format
   if ($newrev[0,6] ==  '000000')
-    # A branch is being deleted
-    all_revs    = [$oldrev]
+    # A branch is being deleted. Do not check old commits for DCO signoff!
+    all_revs    = []
   elsif ($oldrev[0,6] ==  '000000')
     if $refname != 'refs/heads/master'
       # A new branch was pushed. Check all new commits in this branch.