Ruff rules for comprehensions and performance

micropip/install.py

@@ -28,7 +28,7 @@ async def install(
         if isinstance(requirements, str):
             requirements = [requirements]
 
-        fetch_kwargs = dict()
+        fetch_kwargs = {}
 
         if credentials:
             fetch_kwargs["credentials"] = credentials
@@ -84,10 +84,9 @@ async def install(
             )
 
         # Now install PyPI packages
-        for wheel in transaction.wheels:
-            # detect whether the wheel metadata is from PyPI or from custom location
-            # wheel metadata from PyPI has SHA256 checksum digest.
-            wheel_promises.append(wheel.install(wheel_base))
+        # detect whether the wheel metadata is from PyPI or from custom location
+        # wheel metadata from PyPI has SHA256 checksum digest.
+        wheel_promises.extend(wheel.install(wheel_base) for wheel in transaction.wheels)
 
         await asyncio.gather(*wheel_promises)
 

micropip/package.py

@@ -30,8 +30,7 @@ def format_row(values, widths, filler=""):
 
     rows.append(format_row(headers, col_width))
     rows.append(format_row([""] * len(col_width), col_width, filler="-"))
-    for line in table:
-        rows.append(format_row(line, col_width))
+    rows.extend(format_row(line, col_width) for line in table)
 
     return "\n".join(rows)
 

micropip/transaction.py

@@ -48,9 +48,9 @@ async def gather_requirements(
         self,
         requirements: list[str] | list[Requirement],
     ) -> None:
-        requirement_promises = []
-        for requirement in requirements:
-            requirement_promises.append(self.add_requirement(requirement))
+        requirement_promises = [
+            self.add_requirement(requirement) for requirement in requirements
+        ]
 
         await asyncio.gather(*requirement_promises)
 
@@ -132,7 +132,7 @@ def eval_marker(e: dict[str, str]) -> bool:
             # self.ctx_extras is empty and hence the eval_marker() function
             # will not be called at all.
             if not req.marker.evaluate(self.ctx) and not any(
-                [eval_marker(e) for e in self.ctx_extras]
+                eval_marker(e) for e in self.ctx_extras
             ):
                 return
         # Is some version of this package is already installed?

micropip/wheelinfo.py

@@ -52,7 +52,8 @@ class WheelInfo:
     _metadata: Metadata | None = None  # Wheel metadata.
     _requires: list[Requirement] | None = None  # List of requirements.
 
-    # Path to the .dist-info directory. This is only available after extracting the wheel, i.e. after calling `extract()`.
+    # Path to the .dist-info directory.
+    # This is only available after extracting the wheel, i.e. after calling `extract()`.
     _dist_info: Path | None = None
 
     def __post_init__(self):

pyproject.toml

@@ -39,17 +39,23 @@ build-backend = "setuptools.build_meta"
 write_to = "micropip/_version.py"
 
 [tool.ruff]
-line-length = 122
+exclude = ["micropip/externals"]
+line-length = 120
 lint.select = [
-  "E",     # pycodestyles
-  "W",     # pycodestyles
-  "F",     # pyflakes
   "B",     # bugbear
-  "UP",    # pyupgrade
+  "C4",    # flake8-comprehensions
+  "C90",   # mccabe code complexity
+  "E",     # pycodestyle errors
+  "F",     # pyflakes
+  "G",     # flake8-logging-format
   "I",     # isort
+  "PERF",  # perflint
   "PGH",   # pygrep-hooks
-  "G",     # flake8-logging-format
+  "UP",    # pyupgrade
+  "W",     # pycodestyle whitespace
 ]
+lint.flake8-comprehensions.allow-dict-calls-with-keyword-arguments = true
+lint.mccabe.max-complexity = 13
 target-version = "py312"
 
 [tool.ruff.lint.isort]

tests/test_install.py

@@ -249,7 +249,7 @@ async def test_install_with_credentials(selenium):
     fetch_response_mock = MagicMock()
 
     async def myfunc():
-        return json.dumps(dict())
+        return json.dumps({})
 
     fetch_response_mock.string.side_effect = myfunc
 

tests/test_metadata.py

@@ -35,7 +35,7 @@ def test_Metadata_requires(metadata_path, extras, expected):
     m = Metadata(metadata)
 
     reqs = m.requires(extras)
-    reqs_set = set([r.name for r in reqs])
+    reqs_set = {r.name for r in reqs}
     assert reqs_set == set(expected)