Bump ty from 0.0.24 to 0.0.30 (#850)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Tobias Raabe <raabe@posteo.de>

Author: dependabot[bot]

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(

src/_pytask/data_catalog.py

@@ -125,7 +125,8 @@ def add(self, name: str, node: PNode | PProvisionalNode | Any = None) -> None:
                 )
             else:
                 self._entries[name] = self.default_node(name=name)
-            self.path.joinpath(f"{filename}-node.pkl").write_bytes(  # type: ignore[union-attr]
+            assert self.path is not None
+            self.path.joinpath(f"{filename}-node.pkl").write_bytes(
                 pickle.dumps(self._entries[name])
             )
         elif isinstance(node, (PNode, PProvisionalNode)):

src/_pytask/debugging.py

@@ -76,7 +76,8 @@ def _parse_pdbcls(value: str | None) -> tuple[str, str] | None:
             "Must be like 'IPython.terminal.debugger:TerminalPdb'"
         )
         raise ValueError(msg)
-    return tuple(split)  # type: ignore[return-value]
+    modname, classname = split
+    return modname, classname
 
 
 def _pdbcls_callback(
@@ -383,7 +384,8 @@ def wrapper(*args: Any, **kwargs: Any) -> None:
             console.rule("Traceback", characters=">", style="default")
             console.print(Traceback(exc_info))
 
-            post_mortem(exc_info[2])  # type: ignore[arg-type]
+            assert exc_info[2] is not None
+            post_mortem(exc_info[2])
 
             live_manager.resume()
             capman.resume()

src/_pytask/execute.py

@@ -7,6 +7,7 @@
 import time
 from typing import TYPE_CHECKING
 from typing import Any
+from typing import cast
 
 from rich.text import Text
 
@@ -220,7 +221,7 @@ def pytask_execute_task_setup(session: Session, task: PTask) -> None:  # noqa: C
 
                     # Cast reason since get_node_change_info can return "unchanged"
                     # but we only call this when has_changed is True
-                    reason_typed: ReasonType = reason  # type: ignore[assignment]
+                    reason_typed = cast("ReasonType", reason)
                     change_reasons.append(
                         create_change_reason(
                             node=node,

src/_pytask/hookspecs.py

@@ -180,7 +180,7 @@ def pytask_collect_task_setup(
 @hookspec(firstresult=True)
 def pytask_collect_task(
     session: Session, path: Path | None, name: str, obj: Any
-) -> PTask:  # ty: ignore[empty-body]
+) -> PTask:
     """Collect a single task."""
 
 

src/_pytask/mark/structures.py

@@ -8,6 +8,8 @@
 from _pytask.mark_utils import get_all_marks
 from _pytask.models import CollectionMetadata
 from _pytask.typing import TaskFunction
+from _pytask.typing import attach_task_metadata
+from _pytask.typing import is_task_decorator_target
 from _pytask.typing import is_task_function
 
 if TYPE_CHECKING:
@@ -174,9 +176,10 @@ def store_mark(obj: Callable[..., Any], mark: Mark) -> None:
     if isinstance(obj, TaskFunction):
         obj.pytask_meta.markers = [*get_unpacked_marks(obj), mark]
     else:
-        obj.pytask_meta = CollectionMetadata(  # type: ignore[attr-defined]
-            markers=[mark]
-        )
+        if not is_task_decorator_target(obj):
+            msg = "Marks can only be stored on user-defined callables."
+            raise TypeError(msg)
+        attach_task_metadata(obj, CollectionMetadata(markers=[mark]))
 
 
 _DEPRECATION_DECORATOR = """'@pytask.mark.{}' was removed in pytask v0.5.0. To upgrade \