Use Black code formatter

.flake8

@@ -1,5 +1,5 @@
 [flake8]
 max-line-length = 125
-# These codes tend to disagree with yapf
-ignore = E129,W503
+# Whitespace before ':' (in accordance with Black)
+extend-ignore = E203
 max-complexity = 15

.pylintrc

@@ -2,6 +2,7 @@
 disable=
     bad-continuation,
     bad-option-value,
+    bad-whitespace,  # Black deals with whitespace around operators and brackets
     consider-using-f-string,
     duplicate-code,
     fixme,

Makefile

@@ -14,8 +14,8 @@ test: pytest
 lint: flake8 mypy pylint
 
 reformat:
-	$(PYTHON) -m isort --recursive $(PYTHON_DIRS)
-	yapf --parallel --recursive --in-place .
+	$(PYTHON) -m isort $(PYTHON_DIRS)
+	$(PYTHON) -m black $(PYTHON_DIRS)
 
 validate-style:
 	$(eval CHANGES_BEFORE := $(shell mktemp))
@@ -47,8 +47,18 @@ clean:
 	$(RM) -r rpms
 
 build-dep-fedora:
-	sudo dnf install -y --best --allowerasing python3-devel python3-flake8 python3-requests \
-		tar rpmdevtools python3-mypy python3-types-requests python3-pylint python3-isort python3-pytest
+	sudo dnf install -y --best --allowerasing \
+		black \
+		python3-devel \
+		python3-flake8 \
+		python3-isort \
+		python3-mypy \
+		python3-pylint \
+		python3-pytest \
+		python3-requests \
+		python3-types-requests \
+		rpmdevtools \
+		tar
 
 .PHONY: rpm
 rpm: $(RPM)

aiven/client/argx.py

@@ -9,7 +9,20 @@
 from argparse import Action
 from os import PathLike
 from typing import (
-    Any, Callable, cast, Collection, Dict, List, Mapping, NoReturn, Optional, Sequence, TextIO, Tuple, Type, Union
+    Any,
+    Callable,
+    cast,
+    Collection,
+    Dict,
+    List,
+    Mapping,
+    NoReturn,
+    Optional,
+    Sequence,
+    TextIO,
+    Tuple,
+    Type,
+    Union,
 )
 
 import aiven.client.client
@@ -37,9 +50,9 @@
     __version__ = "UNKNOWN"
 
 # cached_property only exists since python 3.8
-SKIP_EVALUATION_TYPES = (property, )
+SKIP_EVALUATION_TYPES = (property,)
 if (sys.version_info[:2] >= (3, 8)) and hasattr(functools, "cached_property"):
-    SKIP_EVALUATION_TYPES += (functools.cached_property, )
+    SKIP_EVALUATION_TYPES += (functools.cached_property,)
 
 ARG_LIST_PROP = "_arg_list"
 LOG_FORMAT = "%(levelname)s\t%(message)s"
@@ -55,8 +68,9 @@ def _get_help_string(self, action: Action) -> str:
                 argparse.OPTIONAL,
                 argparse.ZERO_OR_MORE,
             ]:
-                if (not isinstance(action.default, bool)
-                    and isinstance(action.default, int)) or (isinstance(action.default, str) and action.default):
+                if (not isinstance(action.default, bool) and isinstance(action.default, int)) or (
+                    isinstance(action.default, str) and action.default
+                ):
                     help_text += " (default: %(default)s)"
         return help_text
 
@@ -66,7 +80,7 @@ class UserError(Exception):
 
 
 def arg(*args: Any, **kwargs: Any) -> Callable:
-    """ Declares an argument of an CLI command.
+    """Declares an argument of an CLI command.
 
     This decorator accepts the same arguments as `argparse.Parser::add_argument`.
 
@@ -161,7 +175,7 @@ def add_cmd(self, func: Callable) -> None:
 
         subparsers = self.subparsers
         for level in range(len(cats)):
-            cat = tuple(cats[:level + 1])
+            cat = tuple(cats[: level + 1])
             if cat not in self._cats:
                 parser = subparsers.add_parser(
                     cat[-1],