Fixed a majority of pep8 errors in gcloud.

gcloud/connection.py

@@ -41,4 +41,3 @@ def http(self):
       if self._credentials:
         self._http = self._credentials.authorize(self._http)
     return self._http
-

gcloud/datastore/__init__.py

@@ -69,6 +69,7 @@ def get_connection(client_email, private_key_path):
       client_email, private_key_path, scope=SCOPE)
   return Connection(credentials=credentials)
 
+
 def get_dataset(dataset_id, client_email, private_key_path):
   """Shortcut method to establish a connection to a particular dataset in the Cloud Datastore.
 

gcloud/datastore/connection.py

@@ -49,7 +49,7 @@ def _request(self, dataset_id, method, data):
     headers = {
         'Content-Type': 'application/x-protobuf',
         'Content-Length': str(len(data)),
-        }
+    }
     headers, content = self.http.request(
         uri=self.build_api_url(dataset_id=dataset_id, method=method),
         method='POST', headers=headers, body=data)
@@ -132,7 +132,8 @@ def begin_transaction(self, dataset_id, serializable=False):
     request = datastore_pb.BeginTransactionRequest()
 
     if serializable:
-      request.isolation_level = datastore_pb.BeginTransactionRequest.SERIALIZABLE
+      request.isolation_level = (
+          datastore_pb.BeginTransactionRequest.SERIALIZABLE)
     else:
       request.isolation_level = datastore_pb.BeginTransactionRequest.SNAPSHOT
 
@@ -202,7 +203,8 @@ def run_query(self, dataset_id, query_pb, namespace=None):
       request.partition_id.namespace = namespace
 
     request.query.CopyFrom(query_pb)
-    response = self._rpc(dataset_id, 'runQuery', request, datastore_pb.RunQueryResponse)
+    response = self._rpc(dataset_id, 'runQuery', request,
+                         datastore_pb.RunQueryResponse)
     return [e.entity for e in response.batch.entity_result]
 
   def lookup(self, dataset_id, key_pbs):

gcloud/datastore/dataset.py

@@ -89,8 +89,10 @@ def get_entities(self, keys):
     # This import is here to avoid circular references.
     from gcloud.datastore.entity import Entity
 
-    entity_pbs = self.connection().lookup(dataset_id=self.id(),
-        key_pbs=[k.to_protobuf() for k in keys])
+    entity_pbs = self.connection().lookup(
+        dataset_id=self.id(),
+        key_pbs=[k.to_protobuf() for k in keys]
+    )
 
     entities = []
     for entity_pb in entity_pbs:

gcloud/datastore/demo/__init__.py

@@ -5,10 +5,11 @@
 __all__ = ['get_dataset', 'CLIENT_EMAIL', 'DATASET_ID', 'PRIVATE_KEY_PATH']
 
 
-CLIENT_EMAIL = '754762820716-gimou6egs2hq1rli7el2t621a1b04t9i@developer.gserviceaccount.com'
+CLIENT_EMAIL = ('754762820716-gimou6egs2hq1rli7el2t621a1b04t9i'
+                '@developer.gserviceaccount.com')
 DATASET_ID = 'gcloud-datastore-demo'
 PRIVATE_KEY_PATH = os.path.join(os.path.dirname(__file__), 'demo.key')
 
 
-def get_dataset(): #pragma NO COVER
+def get_dataset():  # pragma NO COVER
   return datastore.get_dataset(DATASET_ID, CLIENT_EMAIL, PRIVATE_KEY_PATH)

gcloud/datastore/entity.py

@@ -43,19 +43,22 @@ class Entity(dict):
     >>> dataset.entity('MyEntityKind')
     <Entity[{'kind': 'MyEntityKind'}] {}>
 
-  - :func:`gcloud.datastore.dataset.Dataset.get_entity` to retrive an existing entity.
+  - :func:`gcloud.datastore.dataset.Dataset.get_entity`
+    to retrieve an existing entity.
 
     >>> dataset.get_entity(key)
     <Entity[{'kind': 'EntityKind', id: 1234}] {'property': 'value'}>
 
-  You can the set values on the entity just like you would on any other dictionary.
+  You can the set values on the entity
+  just like you would on any other dictionary.
 
   >>> entity['age'] = 20
   >>> entity['name'] = 'JJ'
   >>> entity
   <Entity[{'kind': 'EntityKind', id: 1234}] {'age': 20, 'name': 'JJ'}>
 
-  And you can cast an entity to a regular Python dictionary with the `dict` builtin:
+  And you can cast an entity to a regular Python dictionary
+  with the `dict` builtin:
 
   >>> dict(entity)
   {'age': 20, 'name': 'JJ'}
@@ -68,7 +71,7 @@ def __init__(self, dataset=None, kind=None):
       self._key = None
 
   def dataset(self):
-    """Get the :class:`gcloud.datastore.dataset.Dataset` in which this entity belonds.
+    """Get the :class:`gcloud.datastore.dataset.Dataset` in which this entity belongs.
 
     .. note::
       This is based on the :class:`gcloud.datastore.key.Key` set on the entity.
@@ -121,7 +124,8 @@ def from_key(cls, key):
     :type key: :class:`gcloud.datastore.key.Key`
     :param key: The key for the entity.
 
-    :returns: The :class:`Entity` derived from the :class:`gcloud.datastore.key.Key`.
+    :returns: The :class:`Entity` derived from the
+              :class:`gcloud.datastore.key.Key`.
     """
 
     return cls().key(key)
@@ -135,7 +139,8 @@ def from_protobuf(cls, pb, dataset=None):
     :type key: :class:`gcloud.datastore.datastore_v1_pb2.Entity`
     :param key: The Protobuf representing the entity.
 
-    :returns: The :class:`Entity` derived from the :class:`gcloud.datastore.datastore_v1_pb2.Entity`.
+    :returns: The :class:`Entity` derived from the
+              :class:`gcloud.datastore.datastore_v1_pb2.Entity`.
     """
 
     # This is here to avoid circular imports.
@@ -205,9 +210,10 @@ def delete(self):
     self.dataset().connection().delete_entity(
         dataset_id=self.dataset().id(), key_pb=self.key().to_protobuf())
 
-  def __repr__(self): #pragma NO COVER
+  def __repr__(self):  # pragma NO COVER
     # An entity should have a key all the time (even if it's partial).
     if self.key():
-      return '<Entity%s %s>' % (self.key().path(), super(Entity, self).__repr__())
+      return '<Entity%s %s>' % (self.key().path(),
+                                super(Entity, self).__repr__())
     else:
       return '<Entity %s>' % (super(Entity, self).__repr__())

gcloud/datastore/key.py

@@ -21,7 +21,7 @@ def __init__(self, dataset=None, namespace=None, path=None):
 
     :type path: sequence of dicts
     :param path: Each dict must have keys 'kind' (a string) and optionally
-    'name' (a string) or 'id' (an integer).
+                 'name' (a string) or 'id' (an integer).
     """
     self._dataset = dataset
     self._namespace = namespace
@@ -267,7 +267,7 @@ def id_or_name(self):
     """
     return self.id() or self.name()
 
-  def parent(self):#pragma NO COVER
+  def parent(self):  # pragma NO COVER
     """Getter:  return a new key for the next highest element in path.
 
     :rtype: :class:`gcloud.datastore.key.Key`
@@ -278,5 +278,5 @@ def parent(self):#pragma NO COVER
         return None
     return self.path(self.path()[:-1])
 
-  def __repr__(self): #pragma NO COVER
+  def __repr__(self):  # pragma NO COVER
     return '<Key%s>' % self.path()

gcloud/datastore/query.py

@@ -47,7 +47,7 @@ class Query(object):
       '>': datastore_pb.PropertyFilter.GREATER_THAN,
       '>=': datastore_pb.PropertyFilter.GREATER_THAN_OR_EQUAL,
       '=': datastore_pb.PropertyFilter.EQUAL,
-      }
+  }
   """Mapping of operator strings and their protobuf equivalents."""
 
   def __init__(self, kind=None, dataset=None):

gcloud/datastore/test___init__.py

@@ -1,5 +1,6 @@
 import unittest2
 
+
 class Test_get_connection(unittest2.TestCase):
 
     def _callFUT(self, client_email, private_key_path):
@@ -13,6 +14,7 @@ def test_it(self):
         from gcloud.datastore.connection import Connection
         from gcloud.test_credentials import _Client
         from gcloud.test_credentials import _Monkey
+
         CLIENT_EMAIL = 'phred@example.com'
         PRIVATE_KEY = 'SEEkR1t'
         client = _Client()
@@ -27,7 +29,8 @@ def test_it(self):
                          {'service_account_name': CLIENT_EMAIL,
                           'private_key': PRIVATE_KEY,
                           'scope': SCOPE,
-                         })
+                          })
+
 
 class Test_get_dataset(unittest2.TestCase):
 
@@ -43,6 +46,7 @@ def test_it(self):
         from gcloud.datastore.dataset import Dataset
         from gcloud.test_credentials import _Client
         from gcloud.test_credentials import _Monkey
+
         CLIENT_EMAIL = 'phred@example.com'
         PRIVATE_KEY = 'SEEkR1t'
         DATASET_ID = 'DATASET'
@@ -59,4 +63,4 @@ def test_it(self):
                          {'service_account_name': CLIENT_EMAIL,
                           'private_key': PRIVATE_KEY,
                           'scope': SCOPE,
-                         })
+                          })