Use Black skip-magic-trailing-comma option

jeremyn · Mar 30, 2024 · 5550de0 · 5550de0
1 parent 2425113
commit 5550de0
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 18 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,3 +1,6 @@
+[tool.black]
+skip-magic-trailing-comma=true
+
 [tool.isort]
 profile="black"
 

diff --git a/src/book/book.py b/src/book/book.py
@@ -344,10 +344,7 @@ def from_args(cls, b_dir: Path, dir_vars: DirVars, schema: Schema) -> "Book":
                     )
                 case SchemaItemTypes.String():
                     fields.strings[item.name] = _get_string(
-                        metadata,
-                        item,
-                        metadata_dir,
-                        _str_title_prefix(title.display),
+                        metadata, item, metadata_dir, _str_title_prefix(title.display)
                     )
 
         return cls(title, metadata_dir, fields, files)
diff --git a/src/command.py b/src/command.py
@@ -70,7 +70,7 @@ def __call__(
         for val in vals:
             if not (new_l_dir := Path(val)).is_dir():
                 raise SimpleEbookManagerExit(
-                    f"ERROR: library dir '{new_l_dir}' is not a directory.",
+                    f"ERROR: library dir '{new_l_dir}' is not a directory."
                 )
 
             if new_l_dir in l_dirs:

diff --git a/src/util/schema.py b/src/util/schema.py
@@ -153,8 +153,7 @@ def from_args(cls, schema_dict: JSONType) -> "Schema":
             ]
             if len(items_of_type) > 1:
                 raise _InvalidDuplicatesExit(
-                    _get_type_name(type_no_dup),
-                    [item.name for item in items_of_type],
+                    _get_type_name(type_no_dup), [item.name for item in items_of_type]
                 )
 
         return cls(tuple(schema_items))

diff --git a/tests/test_book.py b/tests/test_book.py
@@ -451,7 +451,7 @@ def test_inline_none(self) -> None:
                 SchemaItemTypes.String(self.name, inline=True),
                 Path("."),
                 self.title_prefix,
-            ),
+            )
         )
 
     def test_non_inline_non_none(self) -> None:
@@ -650,8 +650,7 @@ def test_writing_arg_replace_unicode(self) -> None:
         unicode_b_dir = t_dir / "unicode"
         shutil.copytree(orig_b_dir, unicode_b_dir)
         write_text(
-            get_string_fn(unicode_b_dir, self.desc_fieldname),
-            orig_desc + UNICODE_LINE,
+            get_string_fn(unicode_b_dir, self.desc_fieldname), orig_desc + UNICODE_LINE
         )
 
         # test no change with replace_unicode=False

diff --git a/tests/test_db.py b/tests/test_db.py
@@ -165,14 +165,7 @@ def _test_main(
         db_fn.touch()
 
         main_args = (
-            [
-                Args.DIR_VARS.opt,
-                "name1",
-                ".",
-                Args.DIR_VARS.opt,
-                "name2",
-                ".",
-            ]
+            [Args.DIR_VARS.opt, "name1", ".", Args.DIR_VARS.opt, "name2", "."]
             + [Args.LIBRARY_DIRS.opt]
             + [str(l_dir) for l_dir in self.l_dirs]
             + [Args.USER_SQL_FILE.opt, str(USER_SQL_FN)]