[pre-commit.ci] pre-commit autoupdate (#13306)

updates:
  - [github.com/astral-sh/ruff-pre-commit: v0.9.10 → v0.11.0](astral-sh/ruff-pre-commit@v0.9.10...v0.11.0)
  - [github.com/woodruffw/zizmor-pre-commit: v1.4.1 → v1.5.1](woodruffw/zizmor-pre-commit@v1.4.1...v1.5.1)

* [lint] Put noqa in pyproject.toml or locally if possible

Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
 - repo: https://github.com/astral-sh/ruff-pre-commit
-  rev: "v0.9.10"
+  rev: "v0.11.0"
   hooks:
     - id: ruff
       args: ["--fix"]
@@ -12,7 +12,7 @@ repos:
     -   id: end-of-file-fixer
     -   id: check-yaml
 - repo: https://github.com/woodruffw/zizmor-pre-commit
-  rev: v1.4.1
+  rev: v1.5.1
   hooks:
     - id: zizmor
 -   repo: https://github.com/adamchainz/blacken-docs

pyproject.toml

@@ -167,6 +167,10 @@ lint.per-file-ignores."src/_pytest/_py/**/*.py" = [
 lint.per-file-ignores."src/_pytest/_version.py" = [
     "I001",
 ]
+# can't be disabled on a line-by-line basis in file
+lint.per-file-ignores."testing/code/test_source.py" = [
+    "F841",
+]
 lint.per-file-ignores."testing/python/approx.py" = [
     "B015",
 ]

src/_pytest/raises_group.py

@@ -119,12 +119,9 @@ def _check_raw_type(
 def is_fully_escaped(s: str) -> bool:
     # we know we won't compile with re.VERBOSE, so whitespace doesn't need to be escaped
     metacharacters = "{}()+.*?^$[]"
-
-    for i, c in enumerate(s):
-        if c in metacharacters and (i == 0 or s[i - 1] != "\\"):
-            return False
-
-    return True
+    return not any(
+        c in metacharacters and (i == 0 or s[i - 1] != "\\") for (i, c) in enumerate(s)
+    )
 
 
 def unescape(s: str) -> str:
@@ -1198,8 +1195,7 @@ def possible_match(results: ResultHolder, used: set[int] | None = None) -> bool:
     curr_row = len(used)
     if curr_row == len(results.results):
         return True
-
-    for i, val in enumerate(results.results[curr_row]):
-        if val is None and i not in used and possible_match(results, used | {i}):
-            return True
-    return False
+    return any(
+        val is None and i not in used and possible_match(results, used | {i})
+        for (i, val) in enumerate(results.results[curr_row])
+    )

testing/_py/test_local.py

@@ -215,7 +215,7 @@ def test_visit_filterfunc_is_string(self, path1, fil):
         lst = []
         for i in path1.visit(fil):
             lst.append(i.relto(path1))
-        assert len(lst), 2
+        assert len(lst), 2  # noqa: PLC1802,RUF040
         assert "sampledir" in lst
         assert "otherdir" in lst
 

testing/code/test_source.py

@@ -1,21 +1,19 @@
 # mypy: allow-untyped-defs
-# flake8: noqa
-# disable flake check on this file because some constructs are strange
-# or redundant on purpose and can't be disable on a line-by-line basis
+from __future__ import annotations
+
 import inspect
 import linecache
+from pathlib import Path
 import sys
 import textwrap
-from pathlib import Path
 from typing import Any
-from typing import Dict
 
-import pytest
 from _pytest._code import Code
 from _pytest._code import Frame
 from _pytest._code import getfslineno
 from _pytest._code import Source
 from _pytest.pathlib import import_path
+import pytest
 
 
 def test_source_str_function() -> None:
@@ -464,7 +462,6 @@ def test_comment_in_statement() -> None:
 
 def test_source_with_decorator() -> None:
     """Test behavior with Source / Code().source with regard to decorators."""
-    from _pytest.compat import get_real_func
 
     @pytest.mark.foo
     def deco_mark():