You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2017/10/29 00:39:12 UTC

[whimsy] branch master updated: WHIMSY-153 - Validate names in report section

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 82cff3e  WHIMSY-153 - Validate names in report section
82cff3e is described below

commit 82cff3e2e95cc8ad95fc08b0d1f1be2e25edfbc9
Author: Sebb <se...@apache.org>
AuthorDate: Sun Oct 29 01:39:09 2017 +0100

    WHIMSY-153 - Validate names in report section
---
 lib/whimsy/asf/committee.rb | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lib/whimsy/asf/committee.rb b/lib/whimsy/asf/committee.rb
index b28c131..9b02499 100644
--- a/lib/whimsy/asf/committee.rb
+++ b/lib/whimsy/asf/committee.rb
@@ -351,6 +351,10 @@ module ASF
       report.scan(/^([^\n]+)\n---+\n(.*?)\n\n/m).each do |period, committees|
         committees.scan(/^   [ \t]*(.*)/).each do |committee|
           committee, comment = committee.first.split(/[ \t]+#[ \t]+/,2)
+          unless list.include? committee
+            Wunderbar.warn "Unexpected name '#{committee}' in report section; ignored"
+            next
+          end
           committee = list[committee]
           if comment
             committee.report = "#{period}: #{comment}"

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