whimsical-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ru...@apache.org
Subject [whimsy] branch master updated: minor cleanup
Date Mon, 15 May 2017 12:07:11 GMT
This is an automated email from the ASF dual-hosted git repository.

rubys 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  44ca999   minor cleanup
44ca999 is described below

commit 44ca999a81559d83e4dfed2b9978e7c804e542b6
Author: Sam Ruby <rubys@intertwingly.net>
AuthorDate: Mon May 15 08:04:27 2017 -0400

    minor cleanup
---
 tools/site-scan.rb | 2 +-
 www/site.cgi       | 4 +---
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/tools/site-scan.rb b/tools/site-scan.rb
index 41ac380..bcc5fc1 100755
--- a/tools/site-scan.rb
+++ b/tools/site-scan.rb
@@ -151,7 +151,7 @@ def parse(id, site, name)
   doc.css('a').each do |a|
 
     # check the link targets
-    a_href = a['href'] ? a['href'].strip : '' # empty string won't match and won't cause
NPE
+    a_href = a['href'].to_s.strip
 
     if a_href =~ %r{^https?://(www\.)?apache\.org/?$}
       img = a.at('img')
diff --git a/www/site.cgi b/www/site.cgi
index 8d32ff8..daf0092 100755
--- a/www/site.cgi
+++ b/www/site.cgi
@@ -10,8 +10,7 @@ end
 
 $LOAD_PATH.unshift File.realpath(File.expand_path('../../lib', __FILE__))
 require 'json'
-# needed for _whimsy_header/footer
-require 'whimsy/asf'
+require 'whimsy/asf/themes'
 require 'wunderbar'
 require 'wunderbar/bootstrap'
 require 'wunderbar/jquery/stupidtable'
@@ -54,7 +53,6 @@ def analyze(sites)
     ]
 end
 
-# n = project name
 def label(analysis, links, c, n)
   if not links[c]
     'label-danger'

-- 
To stop receiving notification emails like this one, please contact
['"commits@whimsical.apache.org" <commits@whimsical.apache.org>'].

Mime
View raw message