[Collab-qa-commits] r1435 - ddpo-by-mail

[Raphael Geissert]

Author: atomo64-guest
Date: 2009-04-20 18:28:53 +0000 (Mon, 20 Apr 2009)
New Revision: 1435

Modified:
   ddpo-by-mail/reminder-mail.rb
Log:
White space fixes and minor optimisation


Modified: ddpo-by-mail/reminder-mail.rb
===================================================================
--- ddpo-by-mail/reminder-mail.rb	2009-04-20 05:55:50 UTC (rev 1434)
+++ ddpo-by-mail/reminder-mail.rb	2009-04-20 18:28:53 UTC (rev 1435)
@@ -143,6 +143,10 @@
 puts "Preparing mail for each maintainer..."
 # prepare mail for each maintainer
 (maintainers_pkg.keys + subscribers.keys).sort.uniq.each do |email|
+  if EXC_EMAILS.include?(email)
+    puts "Skipping mail for #{email}, IS EXCLUDED"
+    next
+  end
   mpkgs = {}
   maintained = {}
   mserious = false
@@ -254,9 +258,9 @@
       elsif iss[0] == :lintian
         next if (iss[1] < 1 and iss[2] < 1)
         incptext = true
-        ptext += "= Lintian reports "
+        ptext += "= Lintian reports"
         if iss[1] > 0
-          ptext += "#{iss[1]} error(s)"
+          ptext += " #{iss[1]} error(s)"
         end
         if iss[2]
           ptext += " and" if iss[1] > 0
@@ -284,10 +288,6 @@
     puts "Skipping mail for #{email}, only co-maint pkgs have issues"
     next
   end
-  if EXC_EMAILS.include?(email)
-    puts "Skipping mail for #{email}, IS EXCLUDED"
-    next
-  end
   # add dehs text
   if dehstext != ''
     gtext += dehstext + "\n"
@@ -321,12 +321,12 @@
 what you want.
 
 A more detailed status of your packages is available from the DDPO.
-See: 
+See:
 http://qa.debian.org/developer.php?login=#{email}
 
 Don't hesitate to reply to this mail if you have questions. The wiki
 page will be updated with useful information.
--- 
+--
 DDPOMail, run by #{OWNER}
     EOF