[incubator-ponymail] branch master updated: pylint: lazily evaluate logging parameters

[se...@apache.org]

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/incubator-ponymail.git


The following commit(s) were added to refs/heads/master by this push:
     new 6e5c51f  pylint: lazily evaluate logging parameters
6e5c51f is described below

commit 6e5c51f3ef45175aa17679568cab0b1b9b11ef84
Author: Sebb <se...@apache.org>
AuthorDate: Thu May 24 16:51:20 2018 +0100

    pylint: lazily evaluate logging parameters
---
 tools/archiver.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tools/archiver.py b/tools/archiver.py
index 429fd94..bf914e8 100755
--- a/tools/archiver.py
+++ b/tools/archiver.py
@@ -345,7 +345,7 @@ class Archiver(object): # N.B. Also used by import-mbox.py
                     mid = generators.legacy(msg, body, lid, attachments)
             except Exception as err:
                 if logger:
-                    logger.warn("Could not generate MID: %s" % err)
+                    logger.warn("Could not generate MID: %s", err)
                 mid = pmid
             if 'in-reply-to' in msg_metadata:
                 try:
@@ -478,7 +478,7 @@ class Archiver(object): # N.B. Also used by import-mbox.py
             )
         
         if logger:
-            logger.info("Pony Mail archived message %s successfully" % ojson['mid'])
+            logger.info("Pony Mail archived message %s successfully", ojson['mid'])
         oldrefs = []
         
         # Is this a direct reply to a pony mail email?
@@ -513,7 +513,7 @@ class Archiver(object): # N.B. Also used by import-mbox.py
                             }
                         )
                         if logger:
-                            logger.info("Notification sent to %s for %s" % (cid, mid))
+                            logger.info("Notification sent to %s for %s", cid, mid)
 
         # Are there indirect replies to pony emails?
         if msg_metadata.get('references'):
@@ -548,7 +548,7 @@ class Archiver(object): # N.B. Also used by import-mbox.py
                             }
                         )
                         if logger:
-                            logger.info("Notification sent to %s for %s" % (cid, mid))
+                            logger.info("Notification sent to %s for %s", cid, mid)
         return lid, ojson['mid']
 
     def mbox_source(self, b):

-- 
To stop receiving notification emails like this one, please contact
sebb@apache.org.