Fix the letsencrypt script to actually ignore "no action" notifications

sysadmin/letsencrypt_mail.py

@@ -6,8 +6,6 @@
 
 from enum import Enum
 import fileinput
-import sys
-import select
 
 import drymail  # Vendored
 
@@ -44,16 +42,18 @@ def main():
         elif "unexpected error occurred" in line:
             state = State.error
 
-    if state is not State.no_action:
-        text = "\n".join(text)
-        subject = SUBJECT.format(str(state))
-        message = drymail.Message(
-            sender=("Certbot renewal bot", "notify@dennogumi.org"),
-            receivers=["root"],
-            subject=subject,
-            headers=HEADER,
-            text=text
-        )
+    if state is State.no_action:
+        return
+
+    text = "\n".join(text)
+    subject = SUBJECT.format(str(state))
+    message = drymail.Message(
+        sender=("Certbot renewal bot", "notify@dennogumi.org"),
+        receivers=["root"],
+        subject=subject,
+        headers=HEADER,
+        text=text
+    )
 
     client = drymail.SMTPMailer(
         host='localhost')