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 aa151aae Non-PMCs (and Attic) don't need images
aa151aae is described below

commit aa151aaeb8e3228d706ada2fd97bccac1157f346
Author: Sebb <[email protected]>
AuthorDate: Tue Oct 29 17:19:53 2024 +0000

    Non-PMCs (and Attic) don't need images
---
 lib/whimsy/sitestandards.rb | 7 ++++++-
 tools/site-scan.rb          | 1 +
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/lib/whimsy/sitestandards.rb b/lib/whimsy/sitestandards.rb
index 636fbc1d..53bba677 100644
--- a/lib/whimsy/sitestandards.rb
+++ b/lib/whimsy/sitestandards.rb
@@ -155,7 +155,12 @@ module SiteStandards
   #   - the name of the project
   def label(analysis, links, col, name)
     if not links[col]
-      SITE_FAIL
+      # Non-PMCs don't need images
+      if col == 'image' and (name == 'attic' or links['nonpmc'])
+        SITE_PASS
+      else
+        SITE_FAIL
+      end
     elsif analysis[2].include? col and not analysis[2][col].include? name
       SITE_WARN
     else
diff --git a/tools/site-scan.rb b/tools/site-scan.rb
index 42cff462..f27c258c 100755
--- a/tools/site-scan.rb
+++ b/tools/site-scan.rb
@@ -387,6 +387,7 @@ else
       next unless ARGV.include? committee.name
     end
     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'
     # Don't keep checking unnecessarily

Reply via email to