Add pre-commit, ruff check configuration and Github action

.github/workflows/ruff.yml

@@ -0,0 +1,16 @@
+name: ruff
+
+on: [push, pull_request, workflow_dispatch]
+
+permissions:
+  contents: read
+
+jobs:
+  ruff:
+    name: Check code with ruff
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v5
+      - uses: astral-sh/ruff-action@v3
+        with:
+          args: "check"

.pre-commit-config.yaml

@@ -0,0 +1,8 @@
+repos:
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.12.11
+    hooks:
+      - id: ruff-check
+        name: Run Ruff (lint)
+        args: [--exit-non-zero-on-fix]
+        exclude: ^pyperformance/data-files/

.ruff.toml

@@ -0,0 +1,20 @@
+target-version = "py310"
+
+exclude = [
+    "pyperformance/data-files/",
+]
+
+[lint]
+select = [
+    "E",
+    "F",
+]
+
+ignore = [
+    "E402",  # module level import not at top of file
+    "E501",  # line too long
+    "E701",  # multiple statements on one line (colon)
+    "E722",  # do not use bare 'except'
+    "E741",  # ambiguous variable name
+    "F405"   # name may be undefined, or defined from star imports
+]

pyperformance/_benchmark.py

@@ -218,7 +218,7 @@ def _run_perf_script(python, runscript, runid, *,
             '--output', tmp,
         ]
         if pyperf_opts and '--copy-env' in pyperf_opts:
-            argv, env = _prep_cmd(python, runscript, opts, runid, NOOP)
+            argv, env = _prep_cmd(python, runscript, opts, runid, lambda name: None)
         else:
             opts, inherit_envvar = _resolve_restricted_opts(opts)
             argv, env = _prep_cmd(python, runscript, opts, runid, inherit_envvar)

pyperformance/_benchmark_metadata.py

@@ -194,7 +194,7 @@ def _resolve(project, tool, filename):
         if target is None:
             target = field
         if field == 'url':
-            repo = project.get('urls', {}).get('repository')
+            _repo = project.get('urls', {}).get('repository')
             raise NotImplementedError
         elif not resolved.get(target):
             value = project.get(field)

pyperformance/_manifest.py

@@ -343,13 +343,12 @@ def _parse_metafile(metafile, name):
 
 
 def _parse_groups_section(lines):
-    for name in seclines:
+    for name in lines:
         _utils.check_name(name)
         yield name
 
 
 def _parse_group_section(lines):
-    yielded = False
     for line in lines:
         if line.startswith('-'):
             # Exclude a benchmark or group.
@@ -363,7 +362,6 @@ def _parse_group_section(lines):
             name = line
         _benchmark.check_name(name)
         yield op, name
-        yielded = True
 
 
 def _get_tags(benchmarks):

pyperformance/_pyproject_toml.py

@@ -100,7 +100,7 @@ def load_pyproject_toml(filename, *, name=None, tools=None, requirefiles=True):
 
 def _check_relfile(relname, rootdir, kind):
     if os.path.isabs(relname):
-        raise ValuError(f'{relname!r} is absolute, expected relative')
+        raise ValueError(f'{relname!r} is absolute, expected relative')
     actual = os.path.join(rootdir, relname)
     if kind == 'dir':
         if not os.path.isdir(actual):
@@ -122,11 +122,11 @@ def _check_file_or_text(table, rootdir, requirefiles, extra=None):
 
     if 'file' in table:
         if 'text' in table:
-            raise ValueError(f'"file" and "text" are mutually exclusive')
+            raise ValueError('"file" and "text" are mutually exclusive')
         kind = 'file' if requirefiles else None
         _check_relfile(table['file'], rootdir, kind)
     else:
-        text = table['text']
+        _text = table['text']
         # XXX Validate it?
 
 

pyperformance/_utils.py

@@ -6,7 +6,6 @@
     'check_dir',
     # platform
     'MS_WINDOWS',
-    'run_command',
     # misc
     'check_name',
     'parse_name_pattern',

pyperformance/run.py

@@ -1,6 +1,7 @@
 from collections import namedtuple
 import hashlib
 import json
+import os
 import sys
 import time
 import traceback

pyperformance/tests/test_commands.py

@@ -102,13 +102,13 @@ def div():
             print()
 
         def expect_success(*args):
-            text = self.run_pyperformance(
+            _text = self.run_pyperformance(
                 *args,
                 capture=None,
             )
 
         def expect_failure(*args):
-            text = self.run_pyperformance(
+            _text = self.run_pyperformance(
                 *args,
                 capture=None,
                 exitcode=1,
@@ -148,7 +148,7 @@ def test_run_and_show(self):
         # --debug-single-value: benchmark results don't matter, we only
         # check that running benchmarks don't fail.
         # XXX Capture and check the output.
-        text = self.run_pyperformance(
+        _text = self.run_pyperformance(
             'run',
             '-b', 'all',
             '--debug-single-value',