Enable ruff lint and fix 2 extras logging using + concatenation.

pyodide · Aug 26, 2024 · 21c566c · 21c566c
1 parent 3a5b4df
commit 21c566c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/micropip/install.py b/micropip/install.py
@@ -155,7 +155,7 @@ async def install(
     ]
 
     if package_names:
-        logger.info("Installing collected packages: " + ", ".join(package_names))
+        logger.info("Installing collected packages: %s", ", ".join(package_names))
 
     wheel_promises: list[Coroutine[Any, Any, None] | asyncio.Task[Any]] = []
     # Install built-in packages
@@ -182,6 +182,6 @@ async def install(
     ]
 
     if packages:
-        logger.info("Successfully installed " + ", ".join(packages))
+        logger.info("Successfully installed %s", ", ".join(packages))
 
     importlib.invalidate_caches()
diff --git a/pyproject.toml b/pyproject.toml
@@ -50,6 +50,7 @@ select = [
   "UP",    # pyupgrade
   "I",     # isort
   "PGH",   # pygrep-hooks
+  "G",     # flake8-logging-format
 ]
 # Remove E999 once pattern matching is supported
 # https://github.com/charliermarsh/ruff/issues/282