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

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -30,12 +30,12 @@ repos:
     -   id: rst-inline-touching-normal
     -   id: text-unicode-replacement-char
 -   repo: https://github.com/asottile/reorder_python_imports
-    rev: v3.3.0
+    rev: v3.8.1
     hooks:
     -   id: reorder-python-imports
         args: [--py37-plus, --add-import, 'from __future__ import annotations']
 -   repo: https://github.com/asottile/pyupgrade
-    rev: v2.34.0
+    rev: v2.37.1
     hooks:
     -   id: pyupgrade
         args: [--py37-plus]

src/_pytask/live.py

@@ -19,6 +19,7 @@
 from _pytask.report import ExecutionReport
 from _pytask.session import Session
 from _pytask.shared import get_first_non_none_value
+from rich.box import ROUNDED
 from rich.live import Live
 from rich.status import Status
 from rich.style import Style
@@ -242,7 +243,7 @@ def _generate_table(
         else:
             caption_kwargs = {}
 
-        table = Table(**caption_kwargs)
+        table = Table(**caption_kwargs, box=ROUNDED)
         table.add_column("Task", overflow="fold")
         table.add_column("Outcome")
         for report in relevant_reports:

src/_pytask/resolve_dependencies.py

@@ -79,10 +79,12 @@ def pytask_resolve_dependencies_create_dag(tasks: list[Task]) -> nx.DiGraph:
         dag.add_node(task.name, task=task)
 
         tree_map(lambda x: dag.add_node(x.name, node=x), task.depends_on)
-        tree_map(lambda x: dag.add_edge(x.name, task.name), task.depends_on)
+        tree_map(
+            lambda x: dag.add_edge(x.name, task.name), task.depends_on  # noqa: B023
+        )
 
         tree_map(lambda x: dag.add_node(x.name, node=x), task.produces)
-        tree_map(lambda x: dag.add_edge(task.name, x.name), task.produces)
+        tree_map(lambda x: dag.add_edge(task.name, x.name), task.produces)  # noqa: B023
 
     _check_if_dag_has_cycles(dag)
 

tests/test_execute.py

@@ -2,6 +2,7 @@
 
 import re
 import subprocess
+import sys
 import textwrap
 
 import pytest
@@ -12,6 +13,7 @@
 from pytask import TaskOutcome
 
 
+@pytest.mark.xfail(sys.platform == "win32", reason="See #293.")
 @pytest.mark.end_to_end
 def test_python_m_pytask(tmp_path):
     tmp_path.joinpath("task_module.py").write_text("def task_example(): pass")

tests/test_warnings.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import subprocess
+import sys
 import textwrap
 
 import pytest
@@ -122,6 +123,7 @@ def task_example():
     assert ("warning!!!" in result.output) is not add_config
 
 
+@pytest.mark.xfail(sys.platform == "win32", reason="See #293.")
 @pytest.mark.parametrize("warning", ["DeprecationWarning", "PendingDeprecationWarning"])
 def test_deprecation_warnings_are_not_captured(tmp_path, warning):
     path_to_warn_module = tmp_path.joinpath("warning.py")

tox.ini

@@ -75,6 +75,7 @@ filterwarnings =
     ignore: the imp module is deprecated in favour of importlib
     ignore: Using or importing the ABCs from 'collections' instead of from
     ignore: The (parser|symbol) module is deprecated and will be removed in future
+    ignore: nx\.nx_pydot\.to_pydot depends on the pydot package
 markers =
     wip: Tests that are work-in-progress.
     unit: Flag for unit tests which target mainly a single function.