Respond to review comments

Author: AA-Turner

pyproject.toml

@@ -151,6 +151,7 @@ distlib = "https://bitbucket.org/pypa/distlib/raw/master/LICENSE.txt"
 #
 
 [tool.ruff]
+# Pinned to delay pyupgrade changes (https://github.com/pypa/pip/issues/13236)
 target-version = "py38"  # Pin Ruff to Python 3.8
 src = ["src"]
 line-length = 88

tests/functional/test_uninstall_user.py

@@ -79,7 +79,7 @@ def test_uninstall_from_usersite_with_dist_in_global_site(
     @pytest.mark.xfail(
         sys.platform == "darwin"
         and platform.machine() == "arm64"
-        and sys.version_info[:2] == (3, 9),
+        and sys.version_info[:2] < (3, 10),
         reason="Unexpected egg-link install path",
     )
     def test_uninstall_editable_from_usersite(

tests/lib/compat.py

@@ -2,7 +2,7 @@
 
 import contextlib
 import signal
-from typing import Callable, ContextManager, Iterable, Iterator
+from typing import Callable, ContextManager, Iterator
 
 # Applies on Windows.
 if not hasattr(signal, "pthread_sigmask"):
@@ -14,9 +14,6 @@
     @contextlib.contextmanager
     def blocked_signals() -> Iterator[None]:
         """Block all signals for e.g. starting a worker thread."""
-        # valid_signals() was added in Python 3.8 (and not using it results
-        # in a warning on pthread_sigmask() call)
-        mask: Iterable[int]
         mask = signal.valid_signals()
 
         old_mask = signal.pthread_sigmask(  # type: ignore[attr-defined]