Merge pull request #226 from Lukasa/linting

Add linting to the project.

Author: Lukasa

.travis.yml

@@ -23,6 +23,8 @@ matrix:
 
 install:
   - ".travis/install.sh"
+before_script: "flake8 --max-complexity 15 --exclude 'hyper/packages/*' hyper test"
+
 script:
   - >
       if [[ "$TEST_RELEASE" == true ]]; then

.travis/install.sh

@@ -48,3 +48,4 @@ fi
 
 pip install .
 pip install -r test_requirements.txt
+pip install flake8

hyper/__init__.py

@@ -6,7 +6,7 @@
 A module for providing an abstraction layer over the differences between
 HTTP/1.1 and HTTP/2.
 """
-__version__ = '0.5.0'
+import logging
 
 from .common.connection import HTTPConnection
 from .http20.connection import HTTP20Connection
@@ -16,8 +16,10 @@
 
 # Throw import errors on Python <2.7 and 3.0-3.2.
 import sys as _sys
-if _sys.version_info < (2,7) or (3,0) <= _sys.version_info < (3,3):
-    raise ImportError("hyper only supports Python 2.7 and Python 3.3 or higher.")
+if _sys.version_info < (2, 7) or (3, 0) <= _sys.version_info < (3, 3):
+    raise ImportError(
+        "hyper only supports Python 2.7 and Python 3.3 or higher."
+    )
 
 __all__ = [
     HTTPConnection,
@@ -29,5 +31,6 @@
 ]
 
 # Set default logging handler.
-import logging
 logging.getLogger(__name__).addHandler(logging.NullHandler())
+
+__version__ = '0.5.0'

hyper/cli.py

@@ -111,23 +111,25 @@ def make_troubleshooting_argument(parser):
         help="Do HTTP/2 directly in plaintext: skip plaintext upgrade")
 
 
-def set_url_info(args):
-    def split_host_and_port(hostname):
-        if ':' in hostname:
-            return to_host_port_tuple(hostname, default_port=443)
-        return hostname, None
-
-    class UrlInfo(object):
-        def __init__(self):
-            self.fragment = None
-            self.host = 'localhost'
-            self.netloc = None
-            self.path = '/'
-            self.port = 443
-            self.query = None
-            self.scheme = 'https'
-            self.secure = False
+def split_host_and_port(hostname):
+    if ':' in hostname:
+        return to_host_port_tuple(hostname, default_port=443)
+    return hostname, None
+
+
+class UrlInfo(object):
+    def __init__(self):
+        self.fragment = None
+        self.host = 'localhost'
+        self.netloc = None
+        self.path = '/'
+        self.port = 443
+        self.query = None
+        self.scheme = 'https'
+        self.secure = False
 
+
+def set_url_info(args):
     info = UrlInfo()
     _result = urlsplit(args._url)
     for attr in vars(info).keys():
@@ -167,9 +169,9 @@ def set_request_data(args):
             if i.key:
                 headers[i.key] = i.value
             else:
-                # when overriding a HTTP/2 special header there will be a leading 
-                # colon, which tricks the command line parser into thinking 
-                # the header is empty
+                # when overriding a HTTP/2 special header there will be a
+                # leading colon, which tricks the command line parser into
+                # thinking the header is empty
                 k, v = i.value.split(':', 1)
                 headers[':' + k] = v
         elif i.sep == SEP_QUERY:

hyper/common/bufsocket.py

@@ -13,6 +13,7 @@
 import select
 from .exceptions import ConnectionResetError, LineTooLongError
 
+
 class BufferedSocket(object):
     """
     A buffered socket wrapper.
@@ -137,8 +138,7 @@ def recv(self, amt):
         else:
             should_read = True
 
-        if ((self._remaining_capacity > self._bytes_in_buffer) and
-            (should_read)):
+        if (self._remaining_capacity > self._bytes_in_buffer and should_read):
             count = self._sck.recv_into(self._buffer_view[self._buffer_end:])
 
             # The socket just got closed. We should throw an exception if we
@@ -172,7 +172,6 @@ def fill(self):
 
         return
 
-
     def readline(self):
         """
         Read up to a newline from the network and returns it. The implicit

hyper/common/connection.py

@@ -10,6 +10,7 @@
 from ..http20.connection import HTTP20Connection
 from ..tls import H2_NPN_PROTOCOLS, H2C_PROTOCOL
 
+
 class HTTPConnection(object):
     """
     An object representing a single HTTP connection to a server.
@@ -24,26 +25,27 @@ class HTTPConnection(object):
     :param host: The host to connect to. This may be an IP address or a
         hostname, and optionally may include a port: for example,
         ``'http2bin.org'``, ``'http2bin.org:443'`` or ``'127.0.0.1'``.
-    :param port: (optional) The port to connect to. If not provided and one also
-        isn't provided in the ``host`` parameter, defaults to 80.
+    :param port: (optional) The port to connect to. If not provided and one
+        also isn't provided in the ``host`` parameter, defaults to 80.
     :param secure: (optional) Whether the request should use TLS.
         Defaults to ``False`` for most requests, but to ``True`` for any
         request issued to port 443.
     :param window_manager: (optional) The class to use to manage flow control
         windows. This needs to be a subclass of the
-        :class:`BaseFlowControlManager <hyper.http20.window.BaseFlowControlManager>`.
-        If not provided,
+        :class:`BaseFlowControlManager
+        <hyper.http20.window.BaseFlowControlManager>`. If not provided,
         :class:`FlowControlManager <hyper.http20.window.FlowControlManager>`
         will be used.
     :param enable_push: (optional) Whether the server is allowed to push
         resources to the client (see
         :meth:`get_pushes() <hyper.HTTP20Connection.get_pushes>`).
     :param ssl_context: (optional) A class with custom certificate settings.
         If not provided then hyper's default ``SSLContext`` is used instead.
-    :param proxy_host: (optional) The proxy to connect to.  This can be an IP address
-        or a host name and may include a port.
+    :param proxy_host: (optional) The proxy to connect to.  This can be an IP
+        address or a host name and may include a port.
     :param proxy_port: (optional) The proxy port to connect to. If not provided
-        and one also isn't provided in the ``proxy`` parameter, defaults to 8080.
+        and one also isn't provided in the ``proxy`` parameter, defaults to
+        8080.
     """
     def __init__(self,
                  host,

hyper/common/exceptions.py

@@ -5,6 +5,8 @@
 
 Contains hyper's exceptions.
 """
+
+
 class ChunkedDecodeError(Exception):
     """
     An error was encountered while decoding a chunked response.
@@ -43,6 +45,7 @@ class ConnectionResetError(Exception):
         A HTTP connection was unexpectedly reset.
         """
 
+
 class TLSUpgrade(Exception):
     """
     We upgraded to a new protocol in the NPN/ALPN handshake.
@@ -52,6 +55,7 @@ def __init__(self, negotiated, sock):
         self.negotiated = negotiated
         self.sock = sock
 
+
 class HTTPUpgrade(Exception):
     """
     We upgraded to a new protocol via the HTTP Upgrade response.

hyper/common/util.py

@@ -8,7 +8,6 @@
 from hyper.compat import unicode, bytes, imap
 from ..packages.rfc3986.uri import URIReference
 from ..compat import is_py3
-import re
 
 
 def to_bytestring(element):

hyper/compat.py

@@ -1,7 +1,8 @@
 # -*- coding: utf-8 -*-
+# flake8: noqa
 """
 hyper/compat
-~~~~~~~~~
+~~~~~~~~~~~~
 
 Normalizes the Python 2/3 API for internal use.
 """
@@ -21,6 +22,7 @@
 is_py3 = _ver[0] == 3
 is_py3_3 = is_py3 and _ver[1] == 3
 
+
 @contextmanager
 def ignore_missing():
     try:

hyper/contrib.py

@@ -150,7 +150,6 @@ def get_all(self, name, default=None):
             def getheaders(self, name):
                 return self.get_all(name, [])
 
-
         response.raw._original_response = orig = FakeOriginalResponse(None)
         orig.version = 20
         orig.status = resp.status