Revert "Fix pylint for the main package" - accidental push

This reverts commit d0479d6.

Author: Jon Wayne Parrott

core/.flake8

@@ -1,5 +1,4 @@
 [flake8]
-import-order-style=google
 exclude =
   __pycache__,
   .git,

core/google/__init__.py

@@ -12,8 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-"""Google namespace package."""
-
 try:
     import pkg_resources
     pkg_resources.declare_namespace(__name__)

core/google/cloud/__init__.py

@@ -12,8 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-"""Google Cloud namespace package."""
-
 try:
     import pkg_resources
     pkg_resources.declare_namespace(__name__)

core/google/cloud/_helpers.py

@@ -17,6 +17,7 @@
 This module is not part of the public API surface.
 """
 
+# Avoid the grpc and google.cloud.grpc collision.
 from __future__ import absolute_import
 
 import calendar
@@ -31,8 +32,6 @@
 import google_auth_httplib2
 
 try:
-    # pylint: disable=ungrouped-imports
-    # We must import google.auth.transport.grpc within this try: catch.
     import grpc
     import google.auth.transport.grpc
 except ImportError:  # pragma: NO COVER
@@ -105,7 +104,7 @@ def top(self):
         :rtype: object
         :returns: the top-most item, or None if the stack is empty.
         """
-        if self._stack:
+        if len(self._stack) > 0:
             return self._stack[-1]
 
 

core/google/cloud/_http.py

@@ -279,9 +279,7 @@ def api_request(self, method, path, query_params=None,
             can allow custom behavior, for example, to defer an HTTP request
             and complete initialization of the object at a later time.
 
-        :raises ~google.cloud.exceptions.GoogleCloudError: if the response code
-            is not 200 OK.
-        :raises TypeError: if the response content type is not JSON.
+        :raises: Exception if the response code is not 200 OK.
         :rtype: dict or str
         :returns: The API response payload, either as a raw string or
                   a dictionary if the response is valid JSON.

core/google/cloud/_testing.py

@@ -14,15 +14,17 @@
 
 """Shared testing utilities."""
 
+
+# Avoid the grpc and google.cloud.grpc collision.
 from __future__ import absolute_import
 
 
 class _Monkey(object):
-    """Context-manager for replacing module names in the scope of a test."""
+    # context-manager for replacing module names in the scope of a test.
 
     def __init__(self, module, **kw):
         self.module = module
-        if not kw:  # pragma: NO COVER
+        if len(kw) == 0:  # pragma: NO COVER
             raise ValueError('_Monkey was used with nothing to monkey-patch')
         self.to_restore = {key: getattr(module, key) for key in kw}
         for key, value in kw.items():
@@ -66,12 +68,8 @@ def _tempdir_mgr():
     return _tempdir_mgr
 
 
-# pylint: disable=invalid-name
-# Retain _tempdir as a constant for backwards compatibility despite
-# being an invalid name.
 _tempdir = _tempdir_maker()
 del _tempdir_maker
-# pylint: enable=invalid-name
 
 
 class _GAXBaseAPI(object):
@@ -81,8 +79,7 @@ class _GAXBaseAPI(object):
     def __init__(self, **kw):
         self.__dict__.update(kw)
 
-    @staticmethod
-    def _make_grpc_error(status_code, trailing=None):
+    def _make_grpc_error(self, status_code, trailing=None):
         from grpc._channel import _RPCState
         from google.cloud.exceptions import GrpcRendezvous
 
@@ -114,7 +111,6 @@ def __init__(self, *pages, **kwargs):
         self.page_token = kwargs.get('page_token')
 
     def next(self):
-        """Iterate to the next page."""
         import six
         return six.next(self._pages)
 

core/google/cloud/client.py

@@ -64,7 +64,7 @@ def from_service_account_json(cls, json_credentials_path, *args, **kwargs):
 
         :rtype: :class:`_ClientFactoryMixin`
         :returns: The client created with the retrieved JSON credentials.
-        :raises TypeError: if there is a conflict with the kwargs
+        :raises: :class:`TypeError` if there is a conflict with the kwargs
                  and the credentials created by the factory.
         """
         if 'credentials' in kwargs:

core/google/cloud/future/operation.py

@@ -34,7 +34,7 @@ class Operation(base.PollingFuture):
             initial operation.
         refresh (Callable[[], Operation]): A callable that returns the
             latest state of the operation.
-        cancel (Callable[[], None]): A callable that tries to cancel
+        cancel (Callable[[], None]), A callable that tries to cancel
             the operation.
         result_type (type): The protobuf type for the operation's result.
         metadata_type (type): The protobuf type for the operation's

core/google/cloud/iam.py

@@ -226,14 +226,14 @@ def to_api_repr(self):
         if self.version is not None:
             resource['version'] = self.version
 
-        if self._bindings:
+        if len(self._bindings) > 0:
             bindings = resource['bindings'] = []
             for role, members in sorted(self._bindings.items()):
-                if members:
+                if len(members) > 0:
                     bindings.append(
                         {'role': role, 'members': sorted(set(members))})
 
-            if not bindings:
+            if len(bindings) == 0:
                 del resource['bindings']
 
         return resource

core/google/cloud/iterator.py

@@ -242,8 +242,7 @@ def _page_iter(self, increment):
                           results per page while an items iterator will want
                           to increment per item.
 
-        :rtype: :class:`Page`
-        :returns: pages
+        Yields :class:`Page` instances.
         """
         page = self._next_page()
         while page is not None:
@@ -388,8 +387,6 @@ def _get_next_page_response(self):
 
         :rtype: dict
         :returns: The parsed JSON response of the next page's contents.
-
-        :raises ValueError: If the HTTP method is not ``GET`` or ``POST``.
         """
         params = self._get_query_params()
         if self._HTTP_METHOD == 'GET':