Lint

Author: humitos

readthedocs/core/urls/subdomain.py

@@ -1,5 +1,3 @@
-# -*- coding: utf-8 -*-
-
 """URL configurations for subdomains."""
 from functools import reduce
 from operator import add
@@ -25,7 +23,6 @@
 subdomain_urls = [
     url(r'robots\.txt$', robots_txt, name='robots_txt'),
     url(r'sitemap\.xml$', sitemap_xml, name='sitemap_xml'),
-
     url(
         r'^(?:|projects/(?P<subproject_slug>{project_slug})/)'
         r'page/(?P<filename>.*)$'.format(**pattern_opts),

readthedocs/core/views/serve.py

@@ -1,5 +1,3 @@
-# -*- coding: utf-8 -*-
-
 """
 Doc serving from Python.
 
@@ -265,7 +263,7 @@ def _serve_symlink_docs(request, project, privacy_level, filename=''):
         files_tried.append(os.path.join(basepath, filename))
 
     raise Http404(
-        'File not found. Tried these files: %s' % ','.join(files_tried),
+        'File not found. Tried these files: {}'.format(','.join(files_tried),
     )
 
 
@@ -347,6 +345,7 @@ def sitemap_xml(request, project):
 
     :rtype: django.http.HttpResponse
     """
+
     def priorities_generator():
         """
         Generator returning ``priority`` needed by sitemap.xml.
@@ -383,7 +382,10 @@ def changefreqs_generator():
 
     versions = []
     for version, priority, changefreq in zip(
-            sorted_versions, priorities_generator(), changefreqs_generator()):
+            sorted_versions,
+            priorities_generator(),
+            changefreqs_generator(),
+    ):
         element = {
             'loc': version.get_subdomain_url(),
             'priority': priority,
@@ -420,4 +422,9 @@ def changefreqs_generator():
     context = {
         'versions': versions,
     }
-    return render(request, 'sitemap.xml', context, content_type='application/xml')
+    return render(
+        request,
+        'sitemap.xml',
+        context,
+        content_type='application/xml',
+    )

readthedocs/rtd_tests/tests/test_doc_serving.py

@@ -1,21 +1,20 @@
-# -*- coding: utf-8 -*-
+
+import os
 
 import django_dynamic_fixture as fixture
 import mock
-import os
 from django.conf import settings
 from django.contrib.auth.models import User
 from django.http import Http404
-from django.test import TestCase, RequestFactory
+from django.test import RequestFactory, TestCase
 from django.test.utils import override_settings
 from django.urls import reverse
 from mock import mock_open, patch
 
+from readthedocs.builds.models import Version
 from readthedocs.core.middleware import SubdomainMiddleware
 from readthedocs.core.views import server_error_404_subdomain
 from readthedocs.core.views.serve import _serve_symlink_docs
-from readthedocs.builds.constants import STABLE
-from readthedocs.builds.models import Version
 from readthedocs.projects import constants
 from readthedocs.projects.models import Project
 from readthedocs.rtd_tests.base import RequestFactoryTestMixin