Lint

Author: domdfcoding

sphinx_pyproject/__init__.py

@@ -155,13 +155,13 @@ def __init__(
 			if key not in pep621_config:
 				raise BadConfigError(
 						f"Either {key!r} was not declared in the 'project' table "
-						f"or it was marked as 'dynamic', which is unsupported by 'sphinx-pyproject'."
+						"or it was marked as 'dynamic', which is unsupported by 'sphinx-pyproject'.",
 						)
 
 		if "author" not in pep621_config:
 			raise BadConfigError(
-					f"Either 'authors/maintainers' was not declared in the 'project' table "
-					f"or it was marked as 'dynamic', which is unsupported by 'sphinx-pyproject'."
+					"Either 'authors/maintainers' was not declared in the 'project' table "
+					"or it was marked as 'dynamic', which is unsupported by 'sphinx-pyproject'.",
 					)
 
 		self.name = pep621_config["name"]
@@ -276,7 +276,7 @@ def parse_author(config: Dict[str, TOML_TYPES]) -> str:
 		all_authors = list(filter(bool, all_authors))
 
 		if not all_authors:
-			raise BadConfigError(f"The 'project.authors' key cannot be empty.")
+			raise BadConfigError("The 'project.authors' key cannot be empty.")
 
 		return word_join(all_authors)  # type: ignore
 

tests/test_sphinx_pyproject.py

@@ -73,24 +73,26 @@ def test_parse_our_config(advanced_data_regression: AdvancedDataRegressionFixtur
 				pytest.param(
 						f"{MINIMUM}\n[tool.sphinx-pyproject]\ngithub_username = 'username'",
 						1,
-						id="github_username"
+						id="github_username",
 						),
 				pytest.param(
 						f"{MINIMUM}\n[tool.sphinx-pyproject]\ngithub_repository = 'repo'",
 						1,
-						id="github_repository"
+						id="github_repository",
 						),
 				pytest.param(f"{MINIMUM}\n[tool.sphinx-pyproject]\nlanguage = 'en'", 1, id="language"),
 				pytest.param(
-						f"{MINIMUM}\n[tool.sphinx-pyproject]\nextensions = ['sphinx-toolbox']", 1, id="extensions"
+						f"{MINIMUM}\n[tool.sphinx-pyproject]\nextensions = ['sphinx-toolbox']",
+						1,
+						id="extensions",
 						),
 				pytest.param(
 						f"{MINIMUM}\n[tool.sphinx-pyproject]\nhtml_show_sourcelink = true",
 						1,
-						id="html_show_sourcelink"
+						id="html_show_sourcelink",
 						),
 				pytest.param(MULTIPLE_KEYS, 4, id="multiple_keys"),
-				]
+				],
 		)
 def test_parse_config(
 		tmp_pathplus: PathPlus,
@@ -140,9 +142,10 @@ def test_authors_commas(tmp_pathplus: PathPlus):
 				pytest.param("[project]\nname = 'foo'\ndescription = 'Description'", id="name_description"),
 				pytest.param("[project]\nname = 'foo'\nversion = '1.2.3'", id="name_description"),
 				pytest.param(
-						"[project]\nname = 'foo'\nversion = '1.2.3'\ndescription = 'Description'", id="no_authors"
+						"[project]\nname = 'foo'\nversion = '1.2.3'\ndescription = 'Description'",
+						id="no_authors",
 						),
-				]
+				],
 		)
 def test_missing_keys(tmp_pathplus: PathPlus, config: str):
 	(tmp_pathplus / "pyproject.toml").write_text(config)
@@ -174,10 +177,11 @@ def test_missing_keys(tmp_pathplus: PathPlus, config: str):
 
 
 @pytest.mark.parametrize(
-		"toml", [
+		"toml",
+		[
 				pytest.param(POETRY_AUTHORS, id="authors"),
 				pytest.param(POETRY_MAINTAINERS, id="maintainers"),
-				]
+				],
 		)
 def test_poetry(tmp_pathplus: PathPlus, toml: str):
 	(tmp_pathplus / "pyproject.toml").write_text(toml)