Fix failing pre-commit tests (#4434)

Author: cclauss

.pre-commit-config.yaml

@@ -13,7 +13,7 @@ repos:
       - id: check-toml
         name: Validate pyproject.toml
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.12.9
+    rev: v0.13.3
     hooks:
       - id: ruff
         name: ruff lint
@@ -22,7 +22,7 @@ repos:
       - id: ruff-format
         types: [python]
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.17.1
+    rev: v1.18.2
     hooks:
       - id: mypy
         additional_dependencies:

manim/mobject/graphing/probability.py

@@ -170,7 +170,7 @@ def get_subdivision_braces_and_labels(
             "direction": direction,
             "buff": buff,
         }
-        return VGroup(parts.braces, parts.labels)  # type: ignore[arg-type]
+        return VGroup(parts.braces, parts.labels)
 
     def get_side_braces_and_labels(
         self,

manim/scene/vector_space_scene.py

@@ -971,8 +971,8 @@ def add_transformable_label(
             )
         label_mob.vector = vector  # type: ignore[attr-defined]
         label_mob.kwargs = kwargs  # type: ignore[attr-defined]
-        if "animate" in label_mob.kwargs:  # type: ignore[operator]
-            label_mob.kwargs.pop("animate")  # type: ignore[attr-defined]
+        if "animate" in label_mob.kwargs:
+            label_mob.kwargs.pop("animate")
         self.transformable_labels.append(label_mob)
         return cast(MathTex, label_mob)
 

manim/utils/docbuild/autoaliasattr_directive.py

@@ -6,7 +6,7 @@
 
 from docutils import nodes
 from docutils.parsers.rst import Directive
-from docutils.statemachine import ViewList
+from docutils.statemachine import StringList
 
 from manim.utils.docbuild.module_parsing import parse_module_attributes
 
@@ -143,7 +143,7 @@ def run(self) -> list[nodes.Element]:
                     # Using the `.. class::` directive is CRUCIAL, since
                     # function/method parameters are always annotated via
                     # classes - therefore Sphinx expects a class
-                    unparsed = ViewList(
+                    unparsed = StringList(
                         [
                             f".. class:: {alias_name}",
                             "",
@@ -169,7 +169,9 @@ def run(self) -> list[nodes.Element]:
                         # Add all the lines with 4 spaces behind, to consider all the
                         # documentation as a paragraph INSIDE the `.. class::` block
                         doc_lines = alias_doc.split("\n")
-                        unparsed.extend(ViewList([f"    {line}" for line in doc_lines]))
+                        unparsed.extend(
+                            StringList([f"    {line}" for line in doc_lines])
+                        )
 
                     # Parse the reST text into a fresh container
                     # https://www.sphinx-doc.org/en/master/extdev/markupapi.html#parsing-directive-content-as-rest
@@ -191,7 +193,7 @@ def run(self) -> list[nodes.Element]:
                 # Using the `.. class::` directive is CRUCIAL, since
                 # function/method parameters are always annotated via
                 # classes - therefore Sphinx expects a class
-                unparsed = ViewList(
+                unparsed = StringList(
                     [
                         f".. class:: {name}",
                         "",
@@ -218,7 +220,7 @@ def run(self) -> list[nodes.Element]:
             # Let Sphinx Autosummary do its thing as always
             # Add all the attribute names with 4 spaces behind, so that
             # they're considered as INSIDE the `.. autosummary::` block
-            unparsed = ViewList(
+            unparsed = StringList(
                 [
                     ".. autosummary::",
                     *(f"    {attr}" for attr in module_attrs_list),