From 03c883340b776741791d8e2503f06d958e6a99ed Mon Sep 17 00:00:00 2001
From: Tobias Raabe <raabe@posteo.de>
Date: Sun, 27 Aug 2023 18:19:52 +0200
Subject: [PATCH 1/3] Update pre-commit.

---
 .pre-commit-config.yaml | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index cdc3bb664..a945ade00 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -24,24 +24,24 @@ repos:
     -   id: python-no-log-warn
     -   id: text-unicode-replacement-char
 -   repo: https://github.com/asottile/reorder-python-imports
-    rev: v3.9.0
+    rev: v3.10.0
     hooks:
     -   id: reorder-python-imports
         args: [--py38-plus, --add-import, 'from __future__ import annotations']
 -   repo: https://github.com/asottile/setup-cfg-fmt
-    rev: v2.3.0
+    rev: v2.4.0
     hooks:
     -   id: setup-cfg-fmt
 -   repo: https://github.com/psf/black
-    rev: 23.3.0
+    rev: 23.7.0
     hooks:
     -   id: black
 -   repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.270
+    rev: v0.0.286
     hooks:
       - id: ruff
 -   repo: https://github.com/dosisod/refurb
-    rev: v1.16.0
+    rev: v1.20.0
     hooks:
     -   id: refurb
         args: [--ignore, FURB126]
@@ -52,7 +52,7 @@ repos:
         args: [-vv, --fail-under=75]
         exclude: ^(tests/|docs/|scripts/)
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: 'v1.3.0'
+    rev: 'v1.5.1'
     hooks:
     -   id: mypy
         args: [
@@ -66,7 +66,7 @@ repos:
         ]
         pass_filenames: false
 -   repo: https://github.com/executablebooks/mdformat
-    rev: 0.7.16
+    rev: 0.7.17
     hooks:
     -   id: mdformat
         additional_dependencies: [
@@ -76,7 +76,7 @@ repos:
         args: [--wrap, "88"]
         files: (README\.md)
 -   repo: https://github.com/executablebooks/mdformat
-    rev: 0.7.16
+    rev: 0.7.17
     hooks:
     -   id: mdformat
         additional_dependencies: [
@@ -101,7 +101,7 @@ repos:
                 docs/source/tutorials/set_up_a_project.md
             )$
 -   repo: https://github.com/codespell-project/codespell
-    rev: v2.2.4
+    rev: v2.2.5
     hooks:
     -   id: codespell
         additional_dependencies: [tomli]

From 391ee78eca32adc8f5f0be57241768f38474bb3e Mon Sep 17 00:00:00 2001
From: Tobias Raabe <raabe@posteo.de>
Date: Sun, 27 Aug 2023 18:35:39 +0200
Subject: [PATCH 2/3] Almost fix everything.

---
 pyproject.toml                 |  4 ++++
 src/_pytask/click.py           |  3 ++-
 src/_pytask/collect_utils.py   |  4 ++--
 src/_pytask/config_utils.py    |  4 ++--
 src/_pytask/console.py         |  6 ++++--
 src/_pytask/debugging.py       |  3 ++-
 src/_pytask/git.py             |  4 ++--
 src/_pytask/hookspecs.py       | 12 ++++++------
 src/_pytask/logging.py         |  2 +-
 src/_pytask/mark/__init__.pyi  |  4 ++--
 src/_pytask/mark/expression.py |  2 +-
 src/_pytask/outcomes.py        |  2 +-
 src/_pytask/task_utils.py      |  2 +-
 tests/test_capture.py          |  2 +-
 tests/test_clean.py            |  6 ++++--
 tests/test_cli.py              |  2 +-
 tests/test_mark.py             |  4 +++-
 tests/test_warnings.py         |  2 +-
 18 files changed, 40 insertions(+), 28 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index d494d5b14..b1f287ac2 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -93,3 +93,7 @@ markers = [
 ]
 norecursedirs = [".idea", ".tox"]
 filterwarnings = ["ignore:'@pytask.mark.*. is deprecated:DeprecationWarning"]
+
+
+[tool.refurb]
+python_version = "3.8"
diff --git a/src/_pytask/click.py b/src/_pytask/click.py
index d972cd257..6268f7907 100644
--- a/src/_pytask/click.py
+++ b/src/_pytask/click.py
@@ -5,6 +5,7 @@
 import inspect
 from gettext import gettext as _
 from typing import Any
+from typing import ClassVar
 
 import click
 from _pytask import __version__ as version
@@ -55,7 +56,7 @@ def convert(
 class _OptionHighlighter(RegexHighlighter):
     """A highlighter for help texts."""
 
-    highlights = [
+    highlights: ClassVar = [
         r"(?P<switch>\-\w)\b",
         r"(?P<option>\-\-[\w\-]+)",
         r"\-\-[\w\-]+(?P<metavar>[ |=][\w\.:]+)",
diff --git a/src/_pytask/collect_utils.py b/src/_pytask/collect_utils.py
index fa51bfc5f..f2cdd300c 100644
--- a/src/_pytask/collect_utils.py
+++ b/src/_pytask/collect_utils.py
@@ -196,8 +196,8 @@ def _merge_dictionaries(list_of_dicts: list[dict[Any, Any]]) -> dict[Any, Any]:
     """
     merged_dict = _union_of_dictionaries(list_of_dicts)
 
-    if len(merged_dict) == 1 and isinstance(list(merged_dict)[0], _Placeholder):
-        placeholder, value = list(merged_dict.items())[0]
+    if len(merged_dict) == 1 and isinstance(next(iter(merged_dict)), _Placeholder):
+        placeholder, value = next(iter(merged_dict.items()))
         out = value if placeholder.scalar else {0: value}
     else:
         counter = itertools.count()
diff --git a/src/_pytask/config_utils.py b/src/_pytask/config_utils.py
index e870a519c..c8052dfd4 100644
--- a/src/_pytask/config_utils.py
+++ b/src/_pytask/config_utils.py
@@ -14,8 +14,8 @@ def set_defaults_from_config(
     context: click.Context, param: click.Parameter, value: Any  # noqa: ARG001
 ) -> Path | None:
     """Set the defaults for the command-line interface from the configuration."""
-    # Hack: pytask will later walk through all configuration hooks, even the ones not
-    # related to this command. They might expect the defaults coming from their related
+    # pytask will later walk through all configuration hooks, even the ones not related
+    # to this command. They might expect the defaults coming from their related
     # command-line options during parsing. Here, we add their defaults to the
     # configuration.
     command_option_names = [option.name for option in context.command.params]
diff --git a/src/_pytask/console.py b/src/_pytask/console.py
index 2fce20e1c..461cbb443 100644
--- a/src/_pytask/console.py
+++ b/src/_pytask/console.py
@@ -210,7 +210,9 @@ def create_url_style_for_path(path: Path, edtior_url_scheme: str) -> Style:
     )
 
 
-def _get_file(function: Callable[..., Any], skipped_paths: list[Path] = None) -> Path:
+def _get_file(
+    function: Callable[..., Any], skipped_paths: list[Path] | None = None
+) -> Path:
     """Get path to module where the function is defined.
 
     When the ``pdb`` or ``trace`` mode is activated, every task function is wrapped with
@@ -255,7 +257,7 @@ def unify_styles(*styles: str | Style) -> Style:
 
 def create_summary_panel(
     counts: dict[Enum, int],
-    outcome_enum: type[CollectionOutcome] | type[TaskOutcome],
+    outcome_enum: type[CollectionOutcome | TaskOutcome],
     description_total: str,
 ) -> Panel:
     """Create a summary panel."""
diff --git a/src/_pytask/debugging.py b/src/_pytask/debugging.py
index 22198e521..78f2eb673 100644
--- a/src/_pytask/debugging.py
+++ b/src/_pytask/debugging.py
@@ -7,6 +7,7 @@
 from types import FrameType
 from types import TracebackType
 from typing import Any
+from typing import ClassVar
 from typing import Generator
 from typing import TYPE_CHECKING
 
@@ -111,7 +112,7 @@ class PytaskPDB:
 
     _pluginmanager: pluggy.PluginManager | None = None
     _config: dict[str, Any] | None = None
-    _saved: list[tuple[Any, ...]] = []
+    _saved: ClassVar[list[tuple[Any, ...]]] = []
     _recursive_debug: int = 0
     _wrapped_pdb_cls: tuple[type[pdb.Pdb], type[pdb.Pdb]] | None = None
 
diff --git a/src/_pytask/git.py b/src/_pytask/git.py
index df9ea9a82..bfb74474b 100644
--- a/src/_pytask/git.py
+++ b/src/_pytask/git.py
@@ -15,7 +15,7 @@ def is_git_installed() -> bool:
 
 def cmd_output(*cmd: str, **kwargs: Any) -> tuple[int, str, str]:
     """Execute a command and capture the output."""
-    r = subprocess.run(cmd, capture_output=True, **kwargs)
+    r = subprocess.run(cmd, capture_output=True, check=False, **kwargs)
     stdout = r.stdout.decode() if r.stdout is not None else None
     stderr = r.stderr.decode() if r.stderr is not None else None
     return r.returncode, stdout, stderr
@@ -23,7 +23,7 @@ def cmd_output(*cmd: str, **kwargs: Any) -> tuple[int, str, str]:
 
 def init_repo(path: Path) -> None:
     """Initialize a git repository."""
-    subprocess.run(("git", "init"), cwd=path)
+    subprocess.run(("git", "init"), check=False, cwd=path)
 
 
 def zsplit(s: str) -> list[str]:
diff --git a/src/_pytask/hookspecs.py b/src/_pytask/hookspecs.py
index dfa83f92f..dbd565154 100644
--- a/src/_pytask/hookspecs.py
+++ b/src/_pytask/hookspecs.py
@@ -11,7 +11,7 @@
 from typing import TYPE_CHECKING
 
 import click
-import networkx
+import networkx as nx
 import pluggy
 from _pytask.models import NodeInfo
 from _pytask.node_protocols import MetaNode
@@ -226,7 +226,7 @@ def pytask_dag(session: Session) -> None:
 
 
 @hookspec(firstresult=True)
-def pytask_dag_create_dag(session: Session, tasks: list[Task]) -> networkx.DiGraph:
+def pytask_dag_create_dag(session: Session, tasks: list[Task]) -> nx.DiGraph:
     """Create the DAG.
 
     This hook creates the DAG from tasks, dependencies and products. The DAG can be used
@@ -236,7 +236,7 @@ def pytask_dag_create_dag(session: Session, tasks: list[Task]) -> networkx.DiGra
 
 
 @hookspec
-def pytask_dag_modify_dag(session: Session, dag: networkx.DiGraph) -> None:
+def pytask_dag_modify_dag(session: Session, dag: nx.DiGraph) -> None:
     """Modify the DAG.
 
     This hook allows to make some changes to the DAG before it is validated and tasks
@@ -246,7 +246,7 @@ def pytask_dag_modify_dag(session: Session, dag: networkx.DiGraph) -> None:
 
 
 @hookspec(firstresult=True)
-def pytask_dag_validate_dag(session: Session, dag: networkx.DiGraph) -> None:
+def pytask_dag_validate_dag(session: Session, dag: nx.DiGraph) -> None:
     """Validate the DAG.
 
     This hook validates the DAG. For example, there can be cycles in the DAG if tasks,
@@ -256,7 +256,7 @@ def pytask_dag_validate_dag(session: Session, dag: networkx.DiGraph) -> None:
 
 
 @hookspec
-def pytask_dag_select_execution_dag(session: Session, dag: networkx.DiGraph) -> None:
+def pytask_dag_select_execution_dag(session: Session, dag: nx.DiGraph) -> None:
     """Select the subgraph which needs to be executed.
 
     This hook determines which of the tasks have to be re-run because something has
@@ -267,7 +267,7 @@ def pytask_dag_select_execution_dag(session: Session, dag: networkx.DiGraph) ->
 
 @hookspec(firstresult=True)
 def pytask_dag_has_node_changed(
-    session: Session, dag: networkx.DiGraph, node: MetaNode, task_name: str
+    session: Session, dag: nx.DiGraph, node: MetaNode, task_name: str
 ) -> None:
     """Select the subgraph which needs to be executed.
 
diff --git a/src/_pytask/logging.py b/src/_pytask/logging.py
index ce35032cf..0db95358f 100644
--- a/src/_pytask/logging.py
+++ b/src/_pytask/logging.py
@@ -129,7 +129,7 @@ def _format_duration(duration: float) -> str:
 
 
 def _humanize_time(  # noqa: C901, PLR0912
-    amount: int | float, unit: str, short_label: bool = False
+    amount: float, unit: str, short_label: bool = False
 ) -> list[tuple[float, str]]:
     """Humanize the time.
 
diff --git a/src/_pytask/mark/__init__.pyi b/src/_pytask/mark/__init__.pyi
index eca5e6e3b..3dac4a4ec 100644
--- a/src/_pytask/mark/__init__.pyi
+++ b/src/_pytask/mark/__init__.pyi
@@ -15,14 +15,14 @@ def select_by_mark(session: Session, dag: nx.DiGraph) -> set[str]: ...
 
 class MARK_GEN:  # noqa: N801
     @deprecated(
-        "'@pytask.mark.produces' is deprecated starting pytask v0.4.0 and will be removed in v0.5.0. To upgrade your project to the new syntax, read the tutorial on product and dependencies: https://tinyurl.com/yrezszr4.",  # noqa: E501
+        "'@pytask.mark.produces' is deprecated starting pytask v0.4.0 and will be removed in v0.5.0. To upgrade your project to the new syntax, read the tutorial on product and dependencies: https://tinyurl.com/yrezszr4.",  # noqa: E501, PYI053
         category=DeprecationWarning,
         stacklevel=1,
     )
     @staticmethod
     def produces(objects: PyTree[str | Path]) -> None: ...
     @deprecated(
-        "'@pytask.mark.depends_on' is deprecated starting pytask v0.4.0 and will be removed in v0.5.0. To upgrade your project to the new syntax, read the tutorial on product and dependencies: https://tinyurl.com/yrezszr4.",  # noqa: E501
+        "'@pytask.mark.depends_on' is deprecated starting pytask v0.4.0 and will be removed in v0.5.0. To upgrade your project to the new syntax, read the tutorial on product and dependencies: https://tinyurl.com/yrezszr4.",  # noqa: E501, PYI053
         category=DeprecationWarning,
         stacklevel=1,
     )
diff --git a/src/_pytask/mark/expression.py b/src/_pytask/mark/expression.py
index 108da583f..78880669e 100644
--- a/src/_pytask/mark/expression.py
+++ b/src/_pytask/mark/expression.py
@@ -244,7 +244,7 @@ def evaluate(self, matcher: Callable[[str], bool]) -> bool:
             Whether the expression matches or not.
 
         """
-        ret: bool = eval(  # noqa: PGH001
+        ret: bool = eval(  # noqa: PGH001, S307
             self.code, {"__builtins__": {}}, MatcherAdapter(matcher)
         )
         return ret
diff --git a/src/_pytask/outcomes.py b/src/_pytask/outcomes.py
index 23d6a1520..929482875 100644
--- a/src/_pytask/outcomes.py
+++ b/src/_pytask/outcomes.py
@@ -168,7 +168,7 @@ def style_textonly(self) -> str:
 
 def count_outcomes(
     reports: Sequence[CollectionReport | ExecutionReport],
-    outcome_enum: type[CollectionOutcome] | type[TaskOutcome],
+    outcome_enum: type[CollectionOutcome | TaskOutcome],
 ) -> dict[Enum, int]:
     """Count how often an outcome occurred.
 
diff --git a/src/_pytask/task_utils.py b/src/_pytask/task_utils.py
index 32acd15de..adfd7d200 100644
--- a/src/_pytask/task_utils.py
+++ b/src/_pytask/task_utils.py
@@ -118,7 +118,7 @@ def parse_collected_tasks_with_task_marker(
             for unique_name, task in names_to_functions.items():
                 collected_tasks[unique_name] = task
         else:
-            collected_tasks[name] = [i[1] for i in parsed_tasks if i[0] == name][0]
+            collected_tasks[name] = next(i[1] for i in parsed_tasks if i[0] == name)
 
     return collected_tasks
 
diff --git a/tests/test_capture.py b/tests/test_capture.py
index 2b7654a3f..0cda066a8 100644
--- a/tests/test_capture.py
+++ b/tests/test_capture.py
@@ -331,7 +331,7 @@ def test_captureresult() -> None:
     assert err == "err"
     assert cr[0] == "out"
     assert cr[1] == "err"
-    assert cr == cr
+    assert cr == cr  # noqa: PLR0124
     assert cr == CaptureResult("out", "err")
     assert cr != CaptureResult("wrong", "err")
     assert cr == ("out", "err")
diff --git a/tests/test_clean.py b/tests/test_clean.py
index 62a7e6220..a7db90612 100644
--- a/tests/test_clean.py
+++ b/tests/test_clean.py
@@ -71,9 +71,11 @@ def git_project(request, tmp_path):
 
     init_repo(tmp_path)
     subprocess.run(
-        ("git", "add", "task_module.py", "in_tracked.txt", "tracked.txt"), cwd=tmp_path
+        ("git", "add", "task_module.py", "in_tracked.txt", "tracked.txt"),
+        cwd=tmp_path,
+        check=False,
     )
-    subprocess.run(("git", "commit", "-m", "'COMMIT'"), cwd=tmp_path)
+    subprocess.run(("git", "commit", "-m", "'COMMIT'"), cwd=tmp_path, check=False)
 
     return tmp_path
 
diff --git a/tests/test_cli.py b/tests/test_cli.py
index 8e8a076d7..e93938d15 100644
--- a/tests/test_cli.py
+++ b/tests/test_cli.py
@@ -10,7 +10,7 @@
 
 @pytest.mark.end_to_end()
 def test_version_option():
-    process = subprocess.run(["pytask", "--version"], capture_output=True)
+    process = subprocess.run(["pytask", "--version"], capture_output=True, check=False)
     assert "pytask, version " + __version__ in process.stdout.decode("utf-8")
 
 
diff --git a/tests/test_mark.py b/tests/test_mark.py
index 422b7bf09..04c02216b 100644
--- a/tests/test_mark.py
+++ b/tests/test_mark.py
@@ -371,7 +371,9 @@ def task_write_text(depends_on, produces):
     tmp_path.joinpath("in.txt").touch()
 
     result = subprocess.run(
-        ("pytest", tmp_path.joinpath("task_module.py").as_posix()), capture_output=True
+        ("pytest", tmp_path.joinpath("task_module.py").as_posix()),
+        capture_output=True,
+        check=True,
     )
     assert b"DeprecationWarning: '@pytask.mark.depends_on'" in result.stdout
     assert b"DeprecationWarning: '@pytask.mark.produces'" in result.stdout
diff --git a/tests/test_warnings.py b/tests/test_warnings.py
index bb5639979..9d81e21b4 100644
--- a/tests/test_warnings.py
+++ b/tests/test_warnings.py
@@ -155,7 +155,7 @@ def warn_now():
     path_to_warn_module.write_text(textwrap.dedent(warn_module))
 
     # Cannot use runner since then warnings are not ignored by default.
-    result = subprocess.run(("pytask"), cwd=tmp_path, capture_output=True)
+    result = subprocess.run(("pytask"), cwd=tmp_path, capture_output=True, check=True)
 
     assert result.returncode == ExitCode.OK
     assert "Warnings" not in result.stdout.decode()

From 0c5d2d71dff7d2c2493efbfe026fab8bac6617bf Mon Sep 17 00:00:00 2001
From: Tobias Raabe <raabe@posteo.de>
Date: Sun, 27 Aug 2023 18:49:07 +0200
Subject: [PATCH 3/3] Fix all.

---
 src/_pytask/shared.py  | 4 +++-
 tests/test_execute.py  | 2 +-
 tests/test_mark.py     | 2 +-
 tests/test_warnings.py | 2 +-
 4 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/_pytask/shared.py b/src/_pytask/shared.py
index 6059bd1b2..465f15b35 100644
--- a/src/_pytask/shared.py
+++ b/src/_pytask/shared.py
@@ -49,7 +49,9 @@ def parse_paths(x: Any | None) -> list[Path] | None:
     if x is not None:
         paths = [Path(p) for p in to_list(x)]
         paths = [
-            Path(p).resolve() for path in paths for p in glob.glob(path.as_posix())
+            Path(p).resolve()
+            for path in paths
+            for p in glob.glob(path.as_posix())  # noqa: PTH207
         ]
         out = paths
     else:
diff --git a/tests/test_execute.py b/tests/test_execute.py
index 7bddaa416..c9a0f9424 100644
--- a/tests/test_execute.py
+++ b/tests/test_execute.py
@@ -21,7 +21,7 @@
 @pytest.mark.end_to_end()
 def test_python_m_pytask(tmp_path):
     tmp_path.joinpath("task_module.py").write_text("def task_example(): pass")
-    subprocess.run(["python", "-m", "pytask", tmp_path.as_posix()], check=True)
+    subprocess.run(["python", "-m", "pytask", tmp_path.as_posix()], check=False)
 
 
 @pytest.mark.end_to_end()
diff --git a/tests/test_mark.py b/tests/test_mark.py
index 04c02216b..a1fb7773b 100644
--- a/tests/test_mark.py
+++ b/tests/test_mark.py
@@ -373,7 +373,7 @@ def task_write_text(depends_on, produces):
     result = subprocess.run(
         ("pytest", tmp_path.joinpath("task_module.py").as_posix()),
         capture_output=True,
-        check=True,
+        check=False,
     )
     assert b"DeprecationWarning: '@pytask.mark.depends_on'" in result.stdout
     assert b"DeprecationWarning: '@pytask.mark.produces'" in result.stdout
diff --git a/tests/test_warnings.py b/tests/test_warnings.py
index 9d81e21b4..c5b2e763d 100644
--- a/tests/test_warnings.py
+++ b/tests/test_warnings.py
@@ -155,7 +155,7 @@ def warn_now():
     path_to_warn_module.write_text(textwrap.dedent(warn_module))
 
     # Cannot use runner since then warnings are not ignored by default.
-    result = subprocess.run(("pytask"), cwd=tmp_path, capture_output=True, check=True)
+    result = subprocess.run(("pytask"), cwd=tmp_path, capture_output=True, check=False)
 
     assert result.returncode == ExitCode.OK
     assert "Warnings" not in result.stdout.decode()