Fix comma. [skip ci]

Author: tobiasraabe

scripts/update_plugin_list.py

@@ -123,7 +123,7 @@ def _iter_plugins() -> Generator[dict[str, str], None, None]:
         for release in sorted(releases, key=packaging.version.parse, reverse=True):
             if releases[release]:
                 release_date = datetime.date.fromisoformat(
-                    releases[release][-1]["upload_time_iso_8601"].split("T")[0],
+                    releases[release][-1]["upload_time_iso_8601"].split("T")[0]
                 )
                 last_release = release_date.strftime("%b %d, %Y")
                 break
@@ -151,7 +151,7 @@ def _plugin_definitions(plugins: list[dict[str, str]]) -> Generator[str, None, N
                *requires*: {plugin["requires"]}
 
                {plugin["summary"]}
-            """,
+            """
         )
 
 

src/_pytask/build.py

@@ -170,7 +170,7 @@ def main(raw_config: dict[str, Any]) -> Session:  # noqa: C901
     help=("Choose whether tracebacks should be displayed or not."),
 )
 @click.option(
-    "--dry-run", type=bool, is_flag=True, default=False, help="Perform a dry-run.",
+    "--dry-run", type=bool, is_flag=True, default=False, help="Perform a dry-run."
 )
 def build(**raw_config: Any) -> NoReturn:
     """Collect tasks, execute them and report the results.

src/_pytask/capture.py

@@ -156,7 +156,7 @@ class DontReadFromInput:
 
     def read(self, *_args: Any) -> None:
         raise OSError(
-            "pytask: reading from stdin while output is captured! Consider using `-s`.",
+            "pytask: reading from stdin while output is captured! Consider using `-s`."
         )
 
     readline = read
@@ -270,7 +270,7 @@ def _assert_state(self, op: str, states: tuple[str, ...]) -> None:
         assert (
             self._state in states
         ), "cannot {} in state {!r}: expected one of {}".format(
-            op, self._state, ", ".join(states),
+            op, self._state, ", ".join(states)
         )
 
     def start(self) -> None:
@@ -398,7 +398,7 @@ def _assert_state(self, op: str, states: tuple[str, ...]) -> None:
         assert (
             self._state in states
         ), "cannot {} in state {!r}: expected one of {}".format(
-            op, self._state, ", ".join(states),
+            op, self._state, ", ".join(states)
         )
 
     def start(self) -> None:
@@ -512,14 +512,14 @@ def __len__(self) -> int:
     def __iter__(self) -> Iterator[AnyStr]:
         return iter((self.out, self.err))
 
-    def __getitem__(self, item: int) -> AnyStr:
+    def __getitem__(self, item: int) -> AnyStr:  # noqa: ARG002
         return tuple(self)[item]
 
     def _replace(
-        self, *, out: AnyStr | None = None, err: AnyStr | None = None,
+        self, *, out: AnyStr | None = None, err: AnyStr | None = None
     ) -> CaptureResult[AnyStr]:
         return CaptureResult(
-            out=self.out if out is None else out, err=self.err if err is None else err,
+            out=self.out if out is None else out, err=self.err if err is None else err
         )
 
     def count(self, value: AnyStr) -> int:
@@ -634,8 +634,14 @@ 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 ""
+        if self.out:
+            out = self.out.snap()
+        else:
+            out = ""
+        if self.err:
+            err = self.err.snap()
+        else:
+            err = ""
         return CaptureResult(out, err)  # type: ignore
 
 
@@ -654,7 +660,7 @@ def _get_multicapture(method: _CaptureMethod) -> MultiCapture[str]:
         return MultiCapture(in_=None, out=None, err=None)
     if method == _CaptureMethod.TEE_SYS:
         return MultiCapture(
-            in_=None, out=SysCapture(1, tee=True), err=SysCapture(2, tee=True),
+            in_=None, out=SysCapture(1, tee=True), err=SysCapture(2, tee=True)
         )
     raise ValueError(f"unknown capturing method: {method!r}")
 
@@ -681,7 +687,7 @@ def __init__(self, method: _CaptureMethod) -> None:
 
     def __repr__(self) -> str:
         return ("<CaptureManager _method={!r} _capturing={!r}>").format(
-            self._method, self._capturing,
+            self._method, self._capturing
         )
 
     def is_capturing(self) -> bool:

src/_pytask/clean.py

@@ -113,7 +113,7 @@ def clean(**raw_config: Any) -> NoReturn:  # noqa: C901
         session = Session({}, None)
         session.exit_code = ExitCode.CONFIGURATION_FAILED
         exc_info: tuple[
-            type[BaseException], BaseException, TracebackType | None,
+            type[BaseException], BaseException, TracebackType | None
         ] = sys.exc_info()
         console.print(render_exc_info(*exc_info))
 
@@ -126,10 +126,10 @@ def clean(**raw_config: Any) -> NoReturn:  # noqa: C901
             exclude = session.config["exclude"]
             include_directories = session.config["directories"]
             unknown_paths = _find_all_unknown_paths(
-                session, known_paths, exclude, include_directories,
+                session, known_paths, exclude, include_directories
             )
             common_ancestor = find_common_ancestor(
-                *unknown_paths, *session.config["paths"],
+                *unknown_paths, *session.config["paths"]
             )
 
             if unknown_paths:
@@ -145,7 +145,7 @@ def clean(**raw_config: Any) -> NoReturn:  # noqa: C901
                         should_be_deleted = session.config[
                             "mode"
                         ] == _CleanMode.FORCE or click.confirm(
-                            f"Would you like to remove {short_path}?",
+                            f"Would you like to remove {short_path}?"
                         )
                         if should_be_deleted:
                             if not session.config["quiet"]:
@@ -157,7 +157,7 @@ def clean(**raw_config: Any) -> NoReturn:  # noqa: C901
             else:
                 console.print()
                 console.print(
-                    "There are no files and directories which can be deleted.",
+                    "There are no files and directories which can be deleted."
                 )
 
             console.print()
@@ -241,8 +241,8 @@ def _find_all_unknown_paths(
             [
                 _find_all_unkown_paths_per_recursive_node(node, include_directories)
                 for node in recursive_nodes
-            ],
-        ),
+            ]
+        )
     )
     return unknown_paths
 
@@ -269,7 +269,7 @@ class _RecursivePathNode:
 
     @classmethod
     def from_path(
-        cls, path: Path, known_paths: Iterable[Path], exclude: tuple[str, ...],
+        cls, path: Path, known_paths: Iterable[Path], exclude: tuple[str, ...]
     ) -> _RecursivePathNode:
         """Create a node from a path.
 
@@ -310,7 +310,7 @@ def __repr__(self) -> str:
 
 
 def _find_all_unkown_paths_per_recursive_node(
-    node: _RecursivePathNode, include_directories: bool,
+    node: _RecursivePathNode, include_directories: bool
 ) -> Generator[Path, None, None]:
     """Return unknown paths per recursive file node.
 

src/_pytask/cli.py

@@ -44,7 +44,7 @@ def _sort_options_for_each_command_alphabetically(cli: click.Group) -> None:
     """Sort command line options and arguments for each command alphabetically."""
     for command in cli.commands:
         cli.commands[command].params = sorted(
-            cli.commands[command].params, key=lambda x: x.name,
+            cli.commands[command].params, key=lambda x: x.name
         )
 
 
@@ -105,6 +105,7 @@ def pytask_add_hooks(pm: pluggy.PluginManager) -> None:
 @click.version_option(**_VERSION_OPTION_KWARGS)
 def cli() -> None:
     """Manage your tasks with pytask."""
+    pass
 
 
 _extend_command_line_interface(cli)

src/_pytask/click.py

@@ -42,7 +42,7 @@ def __init__(self, enum_type: type[enum.Enum], case_sensitive: bool = True) -> N
         self.enum_type = enum_type
 
     def convert(
-        self, value: Any, param: click.Parameter | None, ctx: click.Context | None,
+        self, value: Any, param: click.Parameter | None, ctx: click.Context | None
     ) -> Any:
         if isinstance(value, enum.Enum):
             value = value.value
@@ -66,17 +66,17 @@ class ColoredGroup(DefaultGroup):
     """A subclass which colors groups with default commands."""
 
     def format_help(
-        self: DefaultGroup, ctx: click.Context, formatter: Any,  # noqa: ARG002
+        self: DefaultGroup, ctx: click.Context, formatter: Any  # noqa: ARG002
     ) -> None:
         """Format the help text."""
         highlighter = _OptionHighlighter()
 
         console.print(
-            f"[b]pytask[/b] [dim]v{version}[/]\n", justify="center", highlight=False,
+            f"[b]pytask[/b] [dim]v{version}[/]\n", justify="center", highlight=False
         )
 
         console.print(
-            "Usage: [b]pytask[/b] [b][OPTIONS][/b] [b][COMMAND][/b] [b][PATHS][/b]\n",
+            "Usage: [b]pytask[/b] [b][OPTIONS][/b] [b][COMMAND][/b] [b][PATHS][/b]\n"
         )
 
         console.print(self.help, style="dim")
@@ -100,7 +100,7 @@ def format_help(
                 title="[bold #f2f2f2]Commands[/]",
                 title_align="left",
                 border_style="grey37",
-            ),
+            )
         )
 
         _print_options(self, ctx)
@@ -115,15 +115,15 @@ class ColoredCommand(click.Command):
     """Override Clicks help with a Richer version."""
 
     def format_help(
-        self: click.Command, ctx: click.Context, formatter: Any,  # noqa: ARG002
+        self: click.Command, ctx: click.Context, formatter: Any  # noqa: ARG002
     ) -> None:
         """Format the help text."""
         console.print(
-            f"[b]pytask[/b] [dim]v{version}[/]\n", justify="center", highlight=False,
+            f"[b]pytask[/b] [dim]v{version}[/]\n", justify="center", highlight=False
         )
 
         console.print(
-            f"Usage: [b]pytask[/b] [b]{self.name}[/b] [b][OPTIONS][/b] [b][PATHS][/b]\n",
+            f"Usage: [b]pytask[/b] [b]{self.name}[/b] [b][OPTIONS][/b] [b][PATHS][/b]\n"
         )
 
         console.print(self.help, style="dim")
@@ -138,7 +138,7 @@ def format_help(
 
 
 def _print_options(
-    group_or_command: click.Command | DefaultGroup, ctx: click.Context,
+    group_or_command: click.Command | DefaultGroup, ctx: click.Context
 ) -> None:
     """Print options formatted with a table in a panel."""
     highlighter = _OptionHighlighter()
@@ -187,7 +187,7 @@ def _print_options(
             title="[bold #f2f2f2]Options[/]",
             title_align="left",
             border_style="grey37",
-        ),
+        )
     )
 
 
@@ -252,7 +252,7 @@ def _format_help_text(param: click.Parameter, ctx: click.Context) -> str:  # noq
             # For boolean flags that have distinct True/False opts,
             # use the opt without prefix instead of the value.
             default_string = split_opt(
-                (param.opts if param.default else param.secondary_opts)[0],
+                (param.opts if param.default else param.secondary_opts)[0]
             )[1]
         elif (
             param.is_bool_flag  # type: ignore[attr-defined]