Lint the site packages in CI and fix what it found

Author: StanFromIreland

.github/workflows/test.yml

@@ -18,7 +18,7 @@ jobs:
         with:
           python-version: "3.13"
       - run: pip install -r requirements-dev.txt
-      - run: flake8 codespeed
+      - run: flake8 codespeed speed_python speed_pypy
 
   test:
     runs-on: ubuntu-latest

codespeed/templates/codespeed/changes_data.html

@@ -23,7 +23,7 @@
     <td>{% if rev.get_browsing_url %}<a href="{{ rev.get_browsing_url }}">{{ rev.commitid }}</a>{% else %}{{ rev.commitid }}{% endif %}</td>
   </tr>
   <tr class="date"><th class="infofirst">Date</td><td>{{ rev.date }}</td></tr>
-  {% if rev.branch.project.repo_type  != "N" %}
+  {% if rev.branch.project.repo_type != "N" %}
     <tr class="repo-path"><th class="infofirst">Repo</th><td>{{ rev.branch.project.repo_path }}</td></tr>
   {% endif %}
 </tbody>

codespeed/templates/codespeed/changes_table.html

@@ -21,7 +21,7 @@
     {% endif%}{% if units.has_stddev %}<td>{{ units.totals.min }}</td>
     {% endif%}{% if units.hasmax %}<td>{{ units.totals.max }}</td>
     {% endif%}<td>{{ units.totals.change|percentage }}</td>
-    <td>{% if units.totals.trend  != "-" %}{{ units.totals.trend|floatformat:2 }}%{% else %}{{ units.totals.trend }}{% endif %}</td>
+    <td>{% if units.totals.trend != "-" %}{{ units.totals.trend|floatformat:2 }}%{% else %}{{ units.totals.trend }}{% endif %}</td>
   </tr>
 </tfoot>
 <tbody>

codespeed/templates/codespeed/timeline.html

@@ -80,7 +80,7 @@
 </div>
 
 <div id="configbar">
-  <span class="options">
+  <span class="options" title="Last {{ defaultlast }} revisions tested">
     Show the last
     <select id="revisions">{% for rev in lastrevisions %}
       <option value="{{ rev }}">{{ rev }}</option>{% endfor %}

speed_pypy/wsgi.py

@@ -1,8 +1,7 @@
 # This is used for staging & production
 import os
-import sys
 
 os.environ.setdefault("DJANGO_SETTINGS_MODULE", "speed_pypy.settings")
 
-from django.core.wsgi import get_wsgi_application
+from django.core.wsgi import get_wsgi_application  # noqa: E402
 application = get_wsgi_application()

speed_python/client.py

@@ -77,7 +77,7 @@ def save_to_speedcenter(url=None, project=None, commitid=None, executable=None,
     parser.add_option("--benchmark")
     parser.add_option("--commitid")
     parser.add_option("--environment",
-        help="Use a custom Codespeed environment")
+                      help="Use a custom Codespeed environment")
     parser.add_option("--executable")
     parser.add_option("--max", type="float")
     parser.add_option("--min", type="float")
@@ -88,7 +88,8 @@ def save_to_speedcenter(url=None, project=None, commitid=None, executable=None,
     parser.add_option("--revision_date")
     parser.add_option("--std-dev", type="float")
     parser.add_option("--url",
-        help="URL of your Codespeed server (e.g. http://codespeed.example.org)")
+                      help="URL of your Codespeed server "
+                           "(e.g. http://codespeed.example.org)")
 
     (options, args) = parser.parse_args()
 
@@ -117,6 +118,6 @@ def save_to_speedcenter(url=None, project=None, commitid=None, executable=None,
     try:
         save_to_speedcenter(**kwargs)
         sys.exit(0)
-    except StandardError, e:
+    except Exception as e:
         logging.error("Error saving results: %s", e)
         sys.exit(1)

speed_python/tests.py

@@ -1,6 +1,7 @@
 """Tests for the speed.python.org specific views.
 
-Run with: python manage.py test codespeed speed_python --settings=codespeed.tests.settings
+Run with:
+    python manage.py test codespeed speed_python --settings=codespeed.tests.settings
 """
 import json
 import os

speed_python/wsgi.py

@@ -1,8 +1,7 @@
 # This is used for staging & production
 import os
-import sys
 
 os.environ.setdefault("DJANGO_SETTINGS_MODULE", "speed_python.settings")
 
-from django.core.wsgi import get_wsgi_application
+from django.core.wsgi import get_wsgi_application  # noqa: E402
 application = get_wsgi_application()