whimsical-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s...@apache.org
Subject [whimsy] branch master updated: notinavail is not a valid id when creating mail addresses
Date Wed, 24 Oct 2018 21:54:30 GMT
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 3dc59ee  notinavail is not a valid id when creating mail addresses
3dc59ee is described below

commit 3dc59eef63d407443f865b560565f9eb079b9894
Author: Sebb <sebb@apache.org>
AuthorDate: Wed Oct 24 22:54:20 2018 +0100

    notinavail is not a valid id when creating mail addresses
---
 lib/whimsy/asf/mail.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/whimsy/asf/mail.rb b/lib/whimsy/asf/mail.rb
index e16065b..f081508 100644
--- a/lib/whimsy/asf/mail.rb
+++ b/lib/whimsy/asf/mail.rb
@@ -165,9 +165,9 @@ module ASF
     end
 
     # All known email addresses: includes active, obsolete, and apache.org
-    # email addresses.
+    # email addresses. (But don't add notinavail@apache.org)
     def all_mail
-      (active_emails + obsolete_emails + ["#{id}@apache.org"]).uniq
+      (active_emails + obsolete_emails + (id == 'notinavail' ? [] : ["#{id}@apache.org"])).uniq
     end
   end
 


Mime
View raw message