(whimsy) branch master updated: Don't unset skipresourcecheck if already true

[sebb]

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 ab61002f Don't unset skipresourcecheck if already true
ab61002f is described below

commit ab61002f79a407ce05ce3e6ed2c9adfd98d76907
Author: Sebb <s...@apache.org>
AuthorDate: Wed Aug 6 23:43:48 2025 +0100

    Don't unset skipresourcecheck if already true
    
    Allow for nil resource
---
 tools/site-scan.rb | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tools/site-scan.rb b/tools/site-scan.rb
index f8e395fd..272b0729 100755
--- a/tools/site-scan.rb
+++ b/tools/site-scan.rb
@@ -389,9 +389,10 @@ else
     results[committee.name] = parse(committee.name, committee.site, 
committee.display_name)
     results[committee.name]['nonpmc'] = committee.nonpmc?
     sites_checked += 1
-    sites_failed += 1 unless results[committee.name][:resources].start_with? 
'Found'
+    sites_failed += 1 unless results[committee.name][:resources]&.start_with? 
'Found'
     # Don't keep checking unnecessarily
-    $skipresourcecheck = (sites_failed > 10 or (sites_failed > 3 and 
sites_failed == sites_checked))
+    # (and don't set false if already true)
+    $skipresourcecheck = ($skipresourcecheck or sites_failed > 10 or 
(sites_failed > 3 and sites_failed == sites_checked))
   end
 
   # Scan podlings that have a website