Enable more lints

List adapted from pytest's configuration.

Author: bluetech

.pre-commit-config.yaml

@@ -11,14 +11,7 @@ repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.5.0
     hooks:
-    -   id: trailing-whitespace
-    -   id: end-of-file-fixer
     -   id: check-yaml
--   repo: https://github.com/asottile/pyupgrade
-    rev: v3.15.1
-    hooks:
-    -   id: pyupgrade
-        args:  [--py38-plus]
 -   repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.2.2
     hooks:

pyproject.toml

@@ -46,8 +46,35 @@ testing = [
 Homepage = "https://execnet.readthedocs.io/en/latest/"
 
 [tool.ruff.lint]
-ignore = ["E741"]
-extend-select = ["I001"]
+extend-select = [
+    "B",    # bugbear
+    "E",    # pycodestyle
+    "F",    # pyflakes
+    "I",    # isort
+    "PYI",  # flake8-pyi
+    "UP",   # pyupgrade
+    "RUF",  # ruff
+    "W",    # pycodestyle
+    "PIE",  # flake8-pie
+    "PGH",  # pygrep-hooks
+    "PLE",  # pylint error
+    "PLW",  # pylint warning
+]
+ignore = [
+    # bugbear ignore
+    "B007",  # Loop control variable `i` not used within loop body
+    "B011",  # Do not `assert False` (`python -O` removes these calls)
+    # pycodestyle ignore
+    "E501",  # Line too long
+    "E741",  # Ambiguous variable name
+    # ruff ignore
+    "RUF012",  # Mutable class attributes should be annotated with `typing.ClassVar`
+    # pylint ignore
+    "PLW0603",  # Using the global statement
+    "PLW0120",  # remove the else and dedent its contents
+    "PLW2901",  # for loop variable overwritten by assignment target
+    "PLR5501",  # Use `elif` instead of `else` then `if`
+]
 
 [tool.ruff.lint.isort]
 force-single-line = true

src/execnet/gateway.py

@@ -143,7 +143,7 @@ def __init__(self, kwargs):
         self.__dict__.update(kwargs)
 
     def __repr__(self):
-        info = ", ".join("%s=%s" % item for item in sorted(self.__dict__.items()))
+        info = ", ".join(f"{k}={v}" for k, v in sorted(self.__dict__.items()))
         return "<RInfo %r>" % info
 
 

src/execnet/gateway_base.py

@@ -542,9 +542,7 @@ def received(self, gateway):
 
     def __repr__(self):
         name = self._types[self.msgcode][0]
-        return "<Message {} channel={} lendata={}>".format(
-            name, self.channelid, len(self.data)
-        )
+        return f"<Message {name} channel={self.channelid} lendata={len(self.data)}>"
 
     def _status(message, gateway):
         # we use the channelid to send back information
@@ -1316,7 +1314,7 @@ def load(self, versioned=False):
                     loader = self.num2func[opcode]
                 except KeyError:
                     raise LoadError(
-                        "unknown opcode %r - wire protocol corruption?" % (opcode,)
+                        f"unknown opcode {opcode!r} - wire protocol corruption?"
                     ) from None
                 loader(self)
         except _Stop:
@@ -1593,7 +1591,7 @@ def _save_integral(self, i, short_op, long_op):
     def save_int(self, i):
         self._save_integral(i, opcode.INT, opcode.LONGINT)
 
-    def save_long(self, l):  # noqa:E741
+    def save_long(self, l):
         self._save_integral(l, opcode.LONG, opcode.LONGLONG)
 
     def save_float(self, flt):

src/execnet/rsync.py

@@ -24,7 +24,7 @@ class RSync:
     def __init__(self, sourcedir, callback=None, verbose=True):
         self._sourcedir = str(sourcedir)
         self._verbose = verbose
-        assert callback is None or hasattr(callback, "__call__")
+        assert callback is None or callable(callback)
         self._callback = callback
         self._channels = {}
         self._receivequeue = Queue()
@@ -168,7 +168,7 @@ def _send_directory(self, path):
                 names.append(name)
                 subpaths.append(p)
         mode = os.lstat(path).st_mode
-        self._broadcast([mode] + names)
+        self._broadcast([mode, *names])
         for p in subpaths:
             self._send_directory_structure(p)
 

src/execnet/rsync_remote.py

@@ -42,7 +42,7 @@ def receive_directory_structure(path, relcomponents):
             entrynames = {}
             for entryname in msg:
                 destpath = os.path.join(path, entryname)
-                receive_directory_structure(destpath, relcomponents + [entryname])
+                receive_directory_structure(destpath, [*relcomponents, entryname])
                 entrynames[entryname] = True
             if options.get("delete"):
                 for othername in os.listdir(path):

src/execnet/script/shell.py

@@ -55,7 +55,7 @@ def run(self):
 
         while 1:
             try:
-                clientfile.write("%s %s >>> " % loc)
+                clientfile.write("{} {} >>> ".format(*loc))
                 clientfile.flush()
                 line = filein.readline()
                 if not line:

src/execnet/script/socketserver.py

@@ -48,7 +48,7 @@ def print_(*args):
 def exec_from_one_connection(serversock):
     print_(progname, "Entering Accept loop", serversock.getsockname())
     clientsock, address = serversock.accept()
-    print_(progname, "got new connection from %s %s" % address)
+    print_(progname, "got new connection from {} {}".format(*address))
     clientfile = clientsock.makefile("rb")
     print_("reading line")
     # rstrip so that we can use \r\n for telnet testing
@@ -60,7 +60,7 @@ def exec_from_one_connection(serversock):
         co = compile(source + "\n", "<socket server>", "exec")
         print_(progname, "compiled source, executing")
         try:
-            exec_(co, g)  # noqa
+            exec_(co, g)  # noqa: F821
         finally:
             print_(progname, "finished executing code")
             # background thread might hold a reference to this (!?)

testing/conftest.py

@@ -152,8 +152,8 @@ def gw(request, execmodel, group):
                 proxygw = group.makegateway("popen//id=%s" % pname)
             # assert group['proxygw'].remote_status().receiving
             gw = group.makegateway(
-                "socket//id=socket//installvia=%s"
-                "//execmodel=%s" % (pname, execmodel.backend)
+                f"socket//id=socket//installvia={pname}"
+                f"//execmodel={execmodel.backend}"
             )
             gw.proxygw = proxygw
             assert pname in group

testing/test_basics.py

@@ -1,3 +1,4 @@
+# ruff: noqa: B018
 from __future__ import annotations
 
 import inspect