Merge remote-tracking branch 'origin/main' into codex/lock-docs

Author: tobiasraabe

.github/workflows/main.yml

@@ -29,11 +29,11 @@ jobs:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
         with:
           persist-credentials: false
-      - uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78 # v7
+      - uses: astral-sh/setup-uv@cec208311dfd045dd5311c1add060b2062131d57 # v8.0.0
         with:
           enable-cache: true
       - name: Install just
-        uses: extractions/setup-just@f8a3cce218d9f83db3a2ecd90e41ac3de6cdfd9b # v3
+        uses: extractions/setup-just@53165ef7e734c5c07cb06b3c8e7b647c5aa16db3 # v4.0.0
       - name: Install graphviz
         run: |
           sudo apt-get update
@@ -57,12 +57,12 @@ jobs:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
         with:
           persist-credentials: false
-      - uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78 # v7
+      - uses: astral-sh/setup-uv@cec208311dfd045dd5311c1add060b2062131d57 # v8.0.0
         with:
           enable-cache: true
           python-version: ${{ matrix.python-version }}
       - name: Install just
-        uses: extractions/setup-just@f8a3cce218d9f83db3a2ecd90e41ac3de6cdfd9b # v3
+        uses: extractions/setup-just@53165ef7e734c5c07cb06b3c8e7b647c5aa16db3 # v4.0.0
 
       - if: matrix.os == 'ubuntu-latest'
         run: |
@@ -81,7 +81,7 @@ jobs:
         run: uv run --group test pytest --cov=src --cov=tests --cov-report=xml -n auto
 
       - name: Upload test coverage reports to Codecov with GitHub Action
-        uses: codecov/codecov-action@75cd11691c0faa626561e295848008c8a7dddffe # v5
+        uses: codecov/codecov-action@57e3a136b779b570ffcdbf80b3bdc90e7fab3de2 # v6.0.0
 
       - name: Run tests with lowest resolution
         if: matrix.python-version == '3.10' && matrix.os == 'ubuntu-latest'

.github/workflows/publish-to-pypi.yml

@@ -16,7 +16,7 @@ jobs:
       with:
         persist-credentials: false
     - name: Install uv
-      uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78 # v7
+      uses: astral-sh/setup-uv@cec208311dfd045dd5311c1add060b2062131d57 # v8.0.0
       with:
         enable-cache: true
         python-version: "3.13"
@@ -47,7 +47,7 @@ jobs:
         name: python-package-distributions
         path: dist/
     - name: Install uv
-      uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78 # v7
+      uses: astral-sh/setup-uv@cec208311dfd045dd5311c1add060b2062131d57 # v8.0.0
     - name: Publish distribution 📦 to PyPI
       run: uv publish
 
@@ -70,7 +70,7 @@ jobs:
         name: python-package-distributions
         path: dist/
     - name: Sign the dists with Sigstore
-      uses: sigstore/gh-action-sigstore-python@a5caf349bc536fbef3668a10ed7f5cd309a4b53d # v3.2.0
+      uses: sigstore/gh-action-sigstore-python@04cffa1d795717b140764e8b640de88853c92acc # v3.3.0
       with:
         inputs: >-
           ./dist/*.tar.gz

.github/workflows/update-plugin-list.yml

@@ -25,7 +25,7 @@ jobs:
           persist-credentials: false
 
       - name: Install uv
-        uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78 # v7
+        uses: astral-sh/setup-uv@cec208311dfd045dd5311c1add060b2062131d57 # v8.0.0
         with:
           python-version: 3.12
 

.github/workflows/zizmor.yml

@@ -26,7 +26,7 @@ jobs:
         with:
           persist-credentials: false
 
-      - uses: astral-sh/setup-uv@37802adc94f370d6bfd71619e3f0bf239e1f3b78 # v7
+      - uses: astral-sh/setup-uv@cec208311dfd045dd5311c1add060b2062131d57 # v8.0.0
         with:
           enable-cache: true
           python-version: '3.13'

.pre-commit-config.yaml

@@ -26,12 +26,12 @@ repos:
     -   id: python-no-log-warn
     -   id: text-unicode-replacement-char
 -   repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.15.8
+    rev: v0.15.10
     hooks:
       - id: ruff-format
       - id: ruff-check
 -   repo: https://github.com/astral-sh/uv-pre-commit
-    rev: 0.11.2
+    rev: 0.11.6
     hooks:
       - id: uv-lock
 -   repo: https://github.com/executablebooks/mdformat
@@ -61,7 +61,7 @@ repos:
     -   id: nbstripout
         exclude: (docs)
 -   repo: https://github.com/crate-ci/typos
-    rev: v1.44.0
+    rev: v1
     hooks:
     -   id: typos
         exclude: (\.ipynb)

pyproject.toml

@@ -171,13 +171,48 @@ unused-ignore-comment = "error"
 [[tool.ty.overrides]]
 include = [
     "src/_pytask/_version.py",
-    "src/_pytask/click.py",
     "tests/test_dag_command.py",
 ]
 
 [tool.ty.overrides.rules]
 unused-ignore-comment = "ignore"
 
+[[tool.ty.overrides]]
+include = ["src/_pytask/click.py"]
+
+[tool.ty.overrides.rules]
+unresolved-attribute = "ignore"
+unused-ignore-comment = "ignore"
+
+[[tool.ty.overrides]]
+include = ["src/_pytask/debugging.py"]
+
+[tool.ty.overrides.rules]
+unsupported-base = "ignore"
+
+[[tool.ty.overrides]]
+include = ["src/_pytask/hookspecs.py"]
+
+[tool.ty.overrides.rules]
+empty-body = "ignore"
+
+[[tool.ty.overrides]]
+include = ["src/_pytask/warnings_utils.py"]
+
+[tool.ty.overrides.rules]
+unresolved-attribute = "ignore"
+
+[[tool.ty.overrides]]
+include = ["tests/**/*.py", "tests/**/*.ipynb"]
+
+[tool.ty.overrides.rules]
+call-non-callable = "ignore"
+invalid-argument-type = "ignore"
+invalid-return-type = "ignore"
+unresolved-attribute = "ignore"
+unresolved-reference = "ignore"
+unsupported-operator = "ignore"
+
 [tool.ty.src]
 include = ["src", "tests"]
 exclude = ["src/_pytask/_hashlib.py"]

src/_pytask/capture.py

@@ -244,7 +244,7 @@ def __exit__(
     @property
     def buffer(self) -> BinaryIO:
         # The str/bytes doesn't actually matter in this type, so OK to fake.
-        return self  # type: ignore[return-value]
+        return cast("BinaryIO", self)
 
 
 # Capture classes.
@@ -553,7 +553,7 @@ def snap(self) -> bytes:
         res = self.tmpfile.buffer.read()
         self.tmpfile.seek(0)
         self.tmpfile.truncate()
-        return res  # type: ignore[return-value]
+        return cast("bytes", res)
 
     def writeorg(self, data: bytes) -> None:
         """Write to original file descriptor."""
@@ -656,8 +656,10 @@ def pop_outerr_to_orig(self) -> tuple[AnyStr, AnyStr]:
         """Pop current snapshot out/err capture and flush to orig streams."""
         out, err = self.readouterr()
         if out:
+            assert self.out is not None
             self.out.writeorg(out)  # type: ignore[union-attr]
         if err:
+            assert self.err is not None
             self.err.writeorg(err)  # type: ignore[union-attr]
         return out, err
 
@@ -678,7 +680,8 @@ def resume_capturing(self) -> None:
         if self.err:
             self.err.resume()
         if self._in_suspended:
-            self.in_.resume()  # type: ignore[union-attr]
+            assert self.in_ is not None
+            self.in_.resume()
             self._in_suspended = False
 
     def stop_capturing(self) -> None:
@@ -699,10 +702,9 @@ def is_started(self) -> bool:
         return self._state == "started"
 
     def readouterr(self) -> CaptureResult[AnyStr]:
-        out = self.out.snap() if self.out else ""
-        err = self.err.snap() if self.err else ""
-        # Will be fixed by pytest. This type error is real, need to fix.
-        return CaptureResult(out, err)  # type: ignore[arg-type]
+        out = self.out.snap() if self.out else cast("AnyStr", "")
+        err = self.err.snap() if self.err else cast("AnyStr", "")
+        return CaptureResult(out, err)
 
 
 def _get_multicapture(method: CaptureMethod) -> MultiCapture[str]:

src/_pytask/collect.py

@@ -192,7 +192,8 @@ def _collect_not_collected_tasks(session: Session) -> None:
     for path in list(COLLECTED_TASKS):
         tasks = COLLECTED_TASKS.pop(path)
         for task in tasks:
-            name = task.pytask_meta.name  # type: ignore[attr-defined]
+            name = task.pytask_meta.name
+            assert name is not None
             node: PTask
             if path:
                 node = Task(base_name=name, path=path, function=task)

src/_pytask/config_utils.py

@@ -43,7 +43,9 @@ def set_defaults_from_config(
     # command-line options during parsing. Here, we add their defaults to the
     # configuration.
     command_option_names = [option.name for option in context.command.params]
-    commands = context.parent.command.commands  # type: ignore[union-attr]
+    assert context.parent is not None
+    assert isinstance(context.parent.command, click.Group)
+    commands = context.parent.command.commands
     all_defaults_from_cli = {
         option.name: option.default
         for name, command in commands.items()

src/_pytask/console.py

@@ -8,6 +8,7 @@
 from pathlib import Path
 from typing import TYPE_CHECKING
 from typing import Any
+from typing import Literal
 from typing import cast
 
 from rich.console import Console
@@ -32,7 +33,6 @@
     from collections.abc import Callable
     from collections.abc import Iterable
     from collections.abc import Sequence
-    from enum import Enum
 
     from _pytask.node_protocols import PTask
     from _pytask.outcomes import CollectionOutcome
@@ -119,7 +119,10 @@ def render_to_string(
         except (AttributeError, IndexError, TypeError):
             theme = None
         render_console = Console(
-            color_system=console.color_system,  # type: ignore[invalid-argument-type]
+            color_system=cast(
+                "Literal['auto', 'standard', '256', 'truecolor', 'windows'] | None",
+                console.color_system,
+            ),
             force_terminal=True,
             width=console.width,
             no_color=False,
@@ -282,7 +285,7 @@ def unify_styles(*styles: str | Style) -> Style:
 
 
 def create_summary_panel(
-    counts: dict[Enum, int],
+    counts: dict[CollectionOutcome | TaskOutcome, int],
     outcome_enum: type[CollectionOutcome | TaskOutcome],
     description_total: str,
 ) -> Panel:
@@ -302,14 +305,14 @@ def create_summary_panel(
             grid.add_row(
                 Padding(str(value), pad=_HORIZONTAL_PADDING),
                 Padding(
-                    outcome.description,  # type: ignore[attr-defined]
+                    outcome.description,
                     pad=_HORIZONTAL_PADDING,
                 ),
                 Padding(
                     percentage,
                     pad=_HORIZONTAL_PADDING,
                 ),
-                style=outcome.style_textonly,  # type: ignore[attr-defined]
+                style=outcome.style_textonly,
             )
 
     return Panel(