Flake8/pep8 the code

pymemcache/client.py

@@ -128,9 +128,9 @@ class Client(object):
      strings must be encoded (as UTF-8, for example) unless they consist only
      of ASCII characters that are neither whitespace nor control characters.
 
-     Values must have a __str__() method to convert themselves to a byte string.
+     Values must have a __str__() method to convert themselves to a byte
-     Unicode objects can be a problem since str() on a Unicode object will
+     string. Unicode objects can be a problem since str() on a Unicode object
-     attempt to encode it as ASCII (which will fail if the value contains
+     will attempt to encode it as ASCII (which will fail if the value contains
      code points larger than U+127). You can fix this will a serializer or by
      just calling encode on the string (using UTF-8, for instance).
 
@@ -602,7 +602,8 @@ class Client(object):
         Args:
           delay: optional int, the number of seconds to wait before flushing,
                  or zero to flush immediately (the default).
-          noreply: optional bool, True to not wait for the response (the default).
+          noreply: optional bool, True to not wait for the response
+                   (the default).
 
         Returns:
           True.

pymemcache/fallback.py

@@ -42,6 +42,7 @@ Best Practices:
    old cluster before you switch away from FallbackClient.
 """
 
+
 class FallbackClient(object):
     def __init__(self, caches):
         assert len(caches) > 0

pymemcache/serde.py

@@ -21,9 +21,9 @@ except ImportError:
     from StringIO import StringIO
 
 
-FLAG_PICKLE  = 1<<0
+FLAG_PICKLE = 1 << 0
-FLAG_INTEGER = 1<<1
+FLAG_INTEGER = 1 << 1
-FLAG_LONG    = 1<<2
+FLAG_LONG = 1 << 2
 
 
 def python_memcache_serializer(key, value):
@@ -46,6 +46,7 @@ def python_memcache_serializer(key, value):
 
     return value, flags
 
+
 def python_memcache_deserializer(key, value, flags):
     if flags == 0:
         return value
@@ -61,7 +62,7 @@ def python_memcache_deserializer(key, value, flags):
             buf = StringIO(value)
             unpickler = pickle.Unpickler(buf)
             return unpickler.load()
-        except Exception as e:
+        except Exception:
             logging.info('Pickle error', exc_info=True)
             return None
 

setup.cfg

@@ -1,2 +1,6 @@
 [wheel]
 universal = true
+
+[flake8]
+exclude = pymemcache/test/
+show-source = True

setup.py

@@ -5,19 +5,19 @@ from setuptools import setup, find_packages
 from pymemcache import __version__
 
 setup(
-    name = 'pymemcache',
+    name='pymemcache',
-    version = __version__,
+    version=__version__,
-    author = 'Charles Gordon',
+    author='Charles Gordon',
-    author_email = 'charles@pinterest.com',
+    author_email='charles@pinterest.com',
-    packages = find_packages(),
+    packages=find_packages(),
-    tests_require = ['nose>=1.0'],
+    tests_require=['nose>=1.0'],
-    test_suite = 'nose.collector',
+    test_suite='nose.collector',
-    install_requires = ['six'],
+    install_requires=['six'],
-    description = 'A comprehensive, fast, pure Python memcached client',
+    description='A comprehensive, fast, pure Python memcached client',
-    long_description = open('README.md').read(),
+    long_description=open('README.md').read(),
-    license = 'Apache License 2.0',
+    license='Apache License 2.0',
-    url = 'https://github.com/Pinterest/pymemcache',
+    url='https://github.com/Pinterest/pymemcache',
-    classifiers = [
+    classifiers=[
         'Programming Language :: Python',
         'Programming Language :: Python :: 2.6',
         'Programming Language :: Python :: 2.7',
@@ -26,4 +26,3 @@ setup(
         'Topic :: Database',
     ],
 )
-

tox.ini

@@ -1,11 +1,17 @@
 [tox]
-envlist = py26, py27, pypy, py33, py34
+envlist = py26, py27, pypy, py33, py34, flake8
 
 [testenv]
 deps = nose
 commands =
     python setup.py nosetests
 
+[testenv:flake8]
+commands =
+	pip install flake8
+	flake8 .
+
+
 [testenv:docs]
 commands =
     pip install -r docs-requirements.txt