Lint

Author: humitos

readthedocs/doc_builder/environments.py

@@ -472,13 +472,12 @@ def run(self, *cmd, **kwargs):
         })
         return super(BuildEnvironment, self).run(*cmd, **kwargs)
 
-    def run_command_class(self, *cmd, **kwargs):
+    def run_command_class(self, *cmd, **kwargs):  # pylint: disable=arguments-differ
         kwargs.update({
             'build_env': self,
         })
         return super(BuildEnvironment, self).run_command_class(*cmd, **kwargs)
 
-
     @property
     def successful(self):
         """Is build completed, without top level failures or failing commands."""  # noqa

readthedocs/projects/models.py

@@ -613,7 +613,6 @@ def vcs_repo(self, version=LATEST, environment=None):
         :param version: version slug for the backend (``LATEST`` by default)
         :type version: str
         """
-
         # TODO: this seems to be the only method that receives a
         # ``version.slug`` instead of a ``Version`` instance (I prefer an
         # instance here)

readthedocs/projects/tasks.py

@@ -65,9 +65,7 @@
 
 class SyncRepositoryMixin(object):
 
-    """
-    Mixin that handles the VCS sync/update.
-    """
+    """Mixin that handles the VCS sync/update."""
 
     @staticmethod
     def get_version(project=None, version_pk=None):
@@ -91,9 +89,7 @@ def get_version(project=None, version_pk=None):
         return APIVersion(**version_data)
 
     def sync_repo(self):
-        """
-        Checkout/update the project's repository and hit ``sync_versions`` API.
-        """
+        """Update the project's repository and hit ``sync_versions`` API."""
         # Make Dirs
         if not os.path.exists(self.project.doc_path):
             os.makedirs(self.project.doc_path)
@@ -158,15 +154,13 @@ def sync_repo(self):
 
 class SyncRepositoryTask(SyncRepositoryMixin, Task):
 
-    """
-    Entry point to synchronize the VCS documentation.
-    """
+    """Entry point to synchronize the VCS documentation."""
 
     max_retries = 5
     default_retry_delay = (7 * 60)
     name = __name__ + '.sync_repository'
 
-    def run(self, version_pk):
+    def run(self, version_pk):  # pylint: disable=arguments-differ
         """
         Run the VCS synchronization.
 
@@ -273,7 +267,6 @@ def run(self, pk, version_pk=None, build_pk=None, record=True,
             if not setup_successful:
                 return False
 
-
         # Catch unhandled errors in the setup step
         except Exception as e:  # noqa
             log.exception(

readthedocs/vcs_support/backends/git.py

@@ -65,16 +65,16 @@ def checkout_revision(self, revision=None):
             branch = self.default_branch or self.fallback_branch
             revision = 'origin/%s' % branch
 
-        code, out, err = self.run('git', 'checkout',
-                           '--force', revision)
+        code, out, err = self.run(
+            'git', 'checkout', '--force', revision)
         if code != 0:
             log.warning("Failed to checkout revision '%s': %s",
                         revision, code)
         return [code, out, err]
 
     def clone(self):
-        code, _, _ = self.run('git', 'clone', '--recursive',
-                           self.repo_url, '.')
+        code, _, _ = self.run(
+            'git', 'clone', '--recursive', self.repo_url, '.')
         if code != 0:
             raise RepositoryError