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

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -9,7 +9,7 @@ repos:
     -   id: debug-statements
     -   id: end-of-file-fixer
 -   repo: https://github.com/pre-commit/pygrep-hooks
-    rev: v1.9.0
+    rev: v1.10.0
     hooks:
     -   id: python-check-blanket-noqa
     -   id: python-check-mock-methods
@@ -36,11 +36,11 @@ repos:
     hooks:
     -   id: black
 -   repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.215
+    rev: v0.0.223
     hooks:
       - id: ruff
 -   repo: https://github.com/dosisod/refurb
-    rev: v1.9.1
+    rev: v1.10.0
     hooks:
     -   id: refurb
         args: [--ignore, FURB126]

pyproject.toml

@@ -51,6 +51,7 @@ extend-ignore = [
     "EM",  # flake8-errmsg
     "ANN401",  # flake8-annotate typing.Any
     "PD",  # pandas-vet
+    "COM812",  # trailing comma missing, but black takes care of that
 ]
 
 

src/pytask_r/collect.py

@@ -18,6 +18,7 @@
 from pytask import Task
 from pytask_r.serialization import SERIALIZERS
 from pytask_r.shared import r
+from pytask_r.shared import R_SCRIPT_KEY
 
 
 def run_r_script(script: Path, options: list[str], serialized: Path) -> None:
@@ -78,14 +79,14 @@ def pytask_collect_task(
         )
 
         if isinstance(task.depends_on, dict):
-            task.depends_on["__script"] = script_node
+            task.depends_on[R_SCRIPT_KEY] = script_node
             task.attributes["r_keep_dict"] = True
         else:
-            task.depends_on = {0: task.depends_on, "__script": script_node}
+            task.depends_on = {0: task.depends_on, R_SCRIPT_KEY: script_node}
             task.attributes["r_keep_dict"] = False
 
         task.function = functools.partial(
-            task.function, script=task.depends_on["__script"].path, options=options
+            task.function, script=task.depends_on[R_SCRIPT_KEY].path, options=options
         )
 
         return task

src/pytask_r/execute.py

@@ -12,6 +12,7 @@
 from pytask_r.serialization import create_path_to_serialized
 from pytask_r.serialization import serialize_keyword_arguments
 from pytask_r.shared import r
+from pytask_r.shared import R_SCRIPT_KEY
 
 
 @hookimpl
@@ -44,13 +45,16 @@ def collect_keyword_arguments(task: Task) -> dict[str, Any]:
     kwargs = dict(task.kwargs)
     task.kwargs = {}
 
-    if len(task.depends_on) == 1 and "__script" in task.depends_on:
+    if len(task.depends_on) == 1 and R_SCRIPT_KEY in task.depends_on:
         pass
-    elif not task.attributes["r_keep_dict"] and len(task.depends_on) == 2:
+    elif (
+        not task.attributes["r_keep_dict"]
+        and len(task.depends_on) == 2  # noqa: PLR2004
+    ):
         kwargs["depends_on"] = str(task.depends_on[0].value)
     else:
         kwargs["depends_on"] = tree_map(lambda x: str(x.value), task.depends_on)
-        kwargs["depends_on"].pop("__script")
+        kwargs["depends_on"].pop(R_SCRIPT_KEY)
 
     if task.produces:
         kwargs["produces"] = tree_map(lambda x: str(x.value), task.produces)

src/pytask_r/parametrize.py

@@ -10,5 +10,5 @@
 @hookimpl
 def pytask_parametrize_kwarg_to_marker(obj: Any, kwargs: dict[Any, Any]) -> None:
     """Attach parametrized r arguments to the function with a marker."""
-    if callable(obj) and "r" in kwargs:
+    if callable(obj) and "r" in kwargs:  # noqa: PLR2004
         pytask.mark.r(**kwargs.pop("r"))(obj)

src/pytask_r/shared.py

@@ -8,6 +8,9 @@
 from typing import Sequence
 
 
+R_SCRIPT_KEY = "__script"
+
+
 def r(
     *,
     script: str | Path,

tests/test_config.py

@@ -8,4 +8,4 @@
 def test_marker_is_configured(tmp_path):
     session = main({"paths": tmp_path})
 
-    assert "r" in session.config["markers"]
+    assert "r" in session.config["markers"]  # noqa: PLR2004

tests/test_execute.py

@@ -256,4 +256,4 @@ def task_run_r_script():
     result = runner.invoke(cli, [tmp_path.as_posix()])
 
     assert result.exit_code == ExitCode.COLLECTION_FAILED
-    assert "has multiple @pytask.mark.r marks" in result.output
+    assert "has multiple @pytask.mark.r marks" in result.output  # noqa: PLR2004