Merge pull request #250 from andriisoldatenko/fix-pep8-errors

Improve code quality

sendgrid/__init__.py

@@ -1,4 +1,4 @@
-from .version import __version__
-#v3 API
-from .sendgrid import SendGridAPIClient
-from .helpers.mail.mail import Email
+from .version import __version__  # noqa
+# v3 API
+from .sendgrid import SendGridAPIClient  # noqa
+from .helpers.mail.mail import Email  # noqa

sendgrid/helpers/inbound/__init__.py

@@ -1,2 +1,2 @@
-from .config import *
-from .parse import *
+from .config import *  # noqa
+from .parse import *  # noqa

sendgrid/helpers/inbound/config.py

@@ -6,12 +6,14 @@
 class Config(object):
     """All configuration for this app is loaded here"""
     def __init__(self, **opts):
-        if (os.environ.get('ENV') != 'prod'):  # We are not in Heroku
+        if os.environ.get('ENV') != 'prod':  # We are not in Heroku
             self.init_environment()
 
         """Allow variables assigned in config.yml available the following variables
            via properties"""
-        self.path = opts.get('path', os.path.abspath(os.path.dirname(__file__)))
+        self.path = opts.get(
+            'path', os.path.abspath(os.path.dirname(__file__))
+        )
         with open(self.path + '/config.yml') as stream:
             config = yaml.load(stream)
             self._debug_mode = config['debug_mode']

sendgrid/helpers/inbound/parse.py

@@ -15,21 +15,25 @@ def __init__(self, config, request):
         self._raw_payload = request.data
 
     def key_values(self):
-        """Return a dictionary of key/values in the payload received from
-           the webhook"""
+        """
+        Return a dictionary of key/values in the payload received from
+        the webhook
+        """
         key_values = {}
         for key in self.keys:
             if key in self.payload:
                 key_values[key] = self.payload[key]
         return key_values
 
     def get_raw_email(self):
-         """This only applies to raw payloads:
-         https://sendgrid.com/docs/Classroom/Basics/Inbound_Parse_Webhook/setting_up_the_inbound_parse_webhook.html#-Raw-Parameters"""
-         if 'email' in self.payload:
+        """
+        This only applies to raw payloads:
+        https://sendgrid.com/docs/Classroom/Basics/Inbound_Parse_Webhook/setting_up_the_inbound_parse_webhook.html#-Raw-Parameters
+        """
+        if 'email' in self.payload:
             raw_email = email.message_from_string(self.payload['email'])
             return raw_email
-         else:
+        else:
             return None
 
     def attachments(self):

sendgrid/helpers/inbound/send.py

@@ -1,11 +1,10 @@
 """A module for sending test SendGrid Inbound Parse messages
 Usage: ./send.py [path to file containing test data]"""
 import argparse
-import os
 import sys
 try:
     from config import Config
-except:
+except ImportError:
     # Python 3+, Travis
     from sendgrid.helpers.inbound.config import Config
 from python_http_client import Client

sendgrid/helpers/mail/__init__.py

@@ -1 +1 @@
-from .mail import *
+from .mail import *  # noqa