Use quotes consistently in log messages

Author: dannyadair

src/pdfbaker/baker.py

@@ -116,7 +116,7 @@ def _teardown_build_directories(self, doc_names: list[str]) -> None:
                 except OSError:
                     # Directory not empty (might contain subdirectories)
                     self.logger.warning(
-                        "Build directory of document not empty, keeping: %s",
+                        "Build directory of document not empty, keeping %s",
                         doc_build_dir,
                     )
 
@@ -127,5 +127,5 @@ def _teardown_build_directories(self, doc_names: list[str]) -> None:
             except OSError:
                 # Directory not empty
                 self.logger.warning(
-                    "Build directory not empty, keeping: %s", self.build_dir
+                    "Build directory not empty, keeping %s", self.build_dir
                 )

src/pdfbaker/common.py

@@ -101,7 +101,7 @@ def combine_pdfs(
                         if str(exc) == "'/Subtype'":
                             # PDF has broken annotations with missing /Subtype
                             logger.warning(
-                                "PDF %s has broken annotations. "
+                                "Broken annotations in PDF: %s"
                                 "Falling back to page-by-page method.",
                                 pdf_file,
                             )

src/pdfbaker/document.py

@@ -168,7 +168,7 @@ def process(self) -> None:
         if "variants" in self.config:
             # Multiple PDF documents
             for variant in self.config["variants"]:
-                self.baker.info("Processing variant: %s", variant["name"])
+                self.baker.info('Processing variant "%s"', variant["name"])
                 variant_config = deep_merge(doc_config, variant)
                 variant_config["variant"] = variant
                 variant_config = resolve_config(variant_config)