fix: reviewer suggestions

Author: ritvi-alagusankar

.github/workflows/test.yml

@@ -335,6 +335,6 @@ jobs:
 
     steps:
     - name: Decide whether the needed jobs succeeded or failed
-      uses: re-actors/alls-green@223e4bb7a751b91f43eda76992bcfbf23b8b0302
+      uses: re-actors/alls-green@2765efec08f0fd63e83ad900f5fd75646be69ff6
       with:
         jobs: ${{ toJSON(needs) }}

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
 - repo: https://github.com/astral-sh/ruff-pre-commit
-  rev: "v0.12.3"
+  rev: "v0.12.5"
   hooks:
     - id: ruff
       args: ["--fix"]
@@ -32,7 +32,7 @@ repos:
     hooks:
     -   id: python-use-type-annotations
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.16.1
+    rev: v1.17.0
     hooks:
     -   id: mypy
         files: ^(src/|testing/|scripts/)

changelog/13621.contrib.rst

@@ -0,0 +1 @@
+pytest's own testsuite now handles the ``lsof`` command hanging (e.g. due to unreachable network filesystems), with the affected selftests being skipped after 10 seconds.

doc/en/reference/plugin_list.rst

@@ -1,7 +1,7 @@
 [build-system]
 build-backend = "setuptools.build_meta"
 requires = [
-    "setuptools>=61",
+    "setuptools>=77",
     "setuptools-scm[toml]>=6.2.3",
 ]
 
@@ -13,7 +13,8 @@ keywords = [
     "test",
     "unittest",
 ]
-license = { text = "MIT" }
+license = "MIT"
+license-files = [ "LICENSE" ]
 authors = [
     { name = "Holger Krekel" },
     { name = "Bruno Oliveira" },
@@ -27,7 +28,6 @@ requires-python = ">=3.9"
 classifiers = [
     "Development Status :: 6 - Mature",
     "Intended Audience :: Developers",
-    "License :: OSI Approved :: MIT License",
     "Operating System :: MacOS",
     "Operating System :: Microsoft :: Windows",
     "Operating System :: POSIX",

pyproject.toml

@@ -62,7 +62,11 @@ def __init__(
         indent: int = 4,
         width: int = 80,
         depth: int | None = None,
+<<<<<<< HEAD
         sort_dicts: bool = False,
+=======
+        sort_dicts: bool = True,
+>>>>>>> 004825824a612277d452488b01af76ce33bb85d9
     ) -> None:
         """Handle pretty printing operations onto a stream using a set of
         configured parameters.

src/_pytest/_io/pprint.py

@@ -407,6 +407,7 @@ class DataclassWithTwoItems:
 def test_consistent_pretty_printer(data: Any, expected: str) -> None:
     assert PrettyPrinter().pformat(data) == textwrap.dedent(expected).strip()
 
+
 @pytest.mark.parametrize(
     ("sort_dicts"),
     (

testing/io/test_pprint.py

@@ -1,6 +1,6 @@
 anyio[trio]==4.9.0
 django==5.2.1
-pytest-asyncio==1.0.0
+pytest-asyncio==1.1.0
 pytest-bdd==8.1.0
 pytest-cov==6.2.1
 pytest-django==4.11.1

testing/plugins_integration/requirements.txt

@@ -983,8 +983,13 @@ def tmpfile(pytester: Pytester) -> Generator[BinaryIO]:
 def lsof_check():
     pid = os.getpid()
     try:
-        out = subprocess.check_output(("lsof", "-p", str(pid))).decode()
-    except (OSError, subprocess.CalledProcessError, UnicodeDecodeError) as exc:
+        out = subprocess.check_output(("lsof", "-p", str(pid)), timeout=10).decode()
+    except (
+        OSError,
+        UnicodeDecodeError,
+        subprocess.CalledProcessError,
+        subprocess.TimeoutExpired,
+    ) as exc:
         # about UnicodeDecodeError, see note on pytester
         pytest.skip(f"could not run 'lsof' ({exc!r})")
     yield