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 2019/09/22 18:02:13 UTC
[whimsy] branch master updated: Move whitelist to more logical place
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 a5707ae Move whitelist to more logical place
a5707ae is described below
commit a5707ae69136182c7fba0acc6723a4f22f42cc79
Author: Sebb <se...@apache.org>
AuthorDate: Sun Sep 22 19:02:09 2019 +0100
Move whitelist to more logical place
---
lib/whimsy/asf/mail.rb | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lib/whimsy/asf/mail.rb b/lib/whimsy/asf/mail.rb
index ef5d9a8..0aa9586 100644
--- a/lib/whimsy/asf/mail.rb
+++ b/lib/whimsy/asf/mail.rb
@@ -87,6 +87,8 @@ module ASF
legal-internal
)
+ WHITELIST = ['infra-users', 'jobs', 'site-dev', 'committers-cvs', 'site-cvs', 'party']
+
# which lists are available for subscription via Whimsy?
# member: true if member
# pmc_chair: true if pmc_chair
@@ -101,9 +103,8 @@ module ASF
lists.delete_if {|list| list =~ /(^|-)security$|^security(-|$)/ }
lists
else
- whitelist = ['infra-users', 'jobs', 'site-dev', 'committers-cvs', 'site-cvs', 'party']
# Can always subscribe to public lists and the whitelist
- lists = @lists.keys.select{|key| @lists[key] == 'public' or whitelist.include? key}
+ lists = @lists.keys.select{|key| @lists[key] == 'public' or WHITELIST.include? key}
# Chairs need the board lists
if pmc_chair