Resolve linting messages in readthedocs.vcs_support.*

Author: gregmuellegger

readthedocs/vcs_support/backends/git.py

@@ -15,7 +15,6 @@ class Backend(BaseVCS):
     supports_branches = True
     fallback_branch = 'master'  # default branch
 
-
     def __init__(self, *args, **kwargs):
         super(Backend, self).__init__(*args, **kwargs)
         self.token = kwargs.get('token', None)
@@ -30,8 +29,9 @@ def _get_clone_url(self):
                 clone_url = 'https://%s@%s' % (self.token, hacked_url)
                 return clone_url
             # Don't edit URL because all hosts aren't the same
-            #else:
-                #clone_url = 'git://%s' % (hacked_url)
+
+            # else:
+                # clone_url = 'git://%s' % (hacked_url)
         return self.repo_url
 
     def set_remote_url(self, url):
@@ -168,7 +168,7 @@ def checkout(self, identifier=None):
 
         identifier = self.find_ref(identifier)
 
-        #Checkout the correct identifier for this branch.
+        # Checkout the correct identifier for this branch.
         code, out, err = self.checkout_revision(identifier)
         if code != 0:
             return code, out, err

readthedocs/vcs_support/backends/svn.py

@@ -37,8 +37,9 @@ def up(self):
                 ("Failed to get code from '%s' (svn revert): %s"
                  % (self.repo_url, retcode))
             )
-        retcode, out, err = self.run('svn', 'up', '--accept', 'theirs-full',
-                           '--trust-server-cert', '--non-interactive')
+        retcode, out, err = self.run(
+            'svn', 'up', '--accept', 'theirs-full',
+            '--trust-server-cert', '--non-interactive')
         if retcode != 0:
             raise ProjectImportError(
                 "Failed to get code from '%s' (svn up): %s" % (self.repo_url,

readthedocs/vcs_support/base.py

@@ -81,9 +81,9 @@ class BaseVCS(BaseCLI):
     supports_tags = False  # Whether this VCS supports tags or not.
     supports_branches = False  # Whether this VCS supports branches or not.
 
-    #==========================================================================
+    # =========================================================================
     # General methods
-    #==========================================================================
+    # =========================================================================
 
     def __init__(self, project, version, **kwargs):
         self.default_branch = project.default_branch
@@ -107,11 +107,11 @@ def update(self):
         """
         self.check_working_dir()
 
-    #==========================================================================
+    # =========================================================================
     # Tag / Branch related methods
     # These methods only apply if supports_tags = True and/or
     # support_branches = True
-    #==========================================================================
+    # =========================================================================
 
     @property
     def tags(self):
@@ -129,15 +129,13 @@ def branches(self):
         """
         raise NotImplementedError
 
-
     @property
     def commit(self):
         """
         Returns a string representing the current commit.
         """
         raise NotImplementedError
 
-
     def checkout(self, identifier=None):
         """
         Set the state to the given identifier.

readthedocs/vcs_support/utils.py

@@ -5,6 +5,7 @@
 
 log = logging.getLogger(__name__)
 
+
 class LockTimeout(Exception):
     pass
 
@@ -94,4 +95,3 @@ def __exit__(self, exc_type, exc_val, exc_tb):
         except (IOError, OSError):
             log.error("Lock (%s): Failed to release, ignoring..." % self.name,
                       exc_info=True)
-