This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new dbcc74a7 A bit more logging
dbcc74a7 is described below

commit dbcc74a7cb2838ffc424910d757513409083cd92
Author: Sebb <[email protected]>
AuthorDate: Thu Mar 7 21:21:14 2024 +0000

    A bit more logging
---
 tools/pubsub-ci-email.rb | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/tools/pubsub-ci-email.rb b/tools/pubsub-ci-email.rb
index d9e75d06..749539e6 100755
--- a/tools/pubsub-ci-email.rb
+++ b/tools/pubsub-ci-email.rb
@@ -149,13 +149,17 @@ def do_diff(initialhash, currenthash, triggerrev)
   end
   #  N.B. before/after are hashes: committee_name => {roster hash}
   ASFJSON.cmphash(before, after) do |bc, type, key, args|
+    # bc = breadcrumb, type = Added/Dropped, key = committeename, args = 
individual roster entry
+    pmcid = bc[1]
+    unless pmcid
+      puts stamp "SKIPPING: #{[bc, type, key, args].inspect} - not a PMC"
+      next
+    end
     unless TYPES.include? type # Don't need to handle changes to entries
-      puts stamp "SKIPPING: #{[bc, type, key, args].inspect}"
+      puts stamp "SKIPPING: #{[bc, type, key, args].inspect} - not a roster 
change"
       next
     end
-    # bc = breadcrumb, type = Added/Dropped, key = committeename, args = 
individual roster entry
-    pmcid = bc[1]
-    next unless pmcid
+    puts stamp "INFO: #{[bc, type, key, args].inspect}"
     cttee = ASF::Committee.find(pmcid)
     ctteename = cttee.display_name
     userid = key

Reply via email to