WIP

Author: jonashaag

pandas/io/formats/excel.py

@@ -85,7 +85,9 @@ def __init__(
         **kwargs,
     ):
         if css_styles and css_converter:
-            css = ";".join(a + ":" + str(v) for (a, v) in css_styles[css_row, css_col])
+            css = ";".join(
+                [a + ":" + str(v) for (a, v) in css_styles[css_row, css_col]]
+            )
             style = css_converter(css)
 
         return super().__init__(row=row, col=col, val=val, style=style, **kwargs)

pandas/io/json/_json.py

@@ -724,7 +724,7 @@ def _combine_lines(self, lines) -> str:
         Combines a list of JSON objects into one JSON object.
         """
         return (
-            f'[{",".join((line for line in (line.strip() for line in lines) if line))}]'
+            f'[{",".join([line for line in (line.strip() for line in lines) if line])}]'
         )
 
     def read(self):

pandas/io/pytables.py

@@ -2040,10 +2040,10 @@ def __repr__(self) -> str:
             map(pprint_thing, (self.name, self.cname, self.axis, self.pos, self.kind))
         )
         return ",".join(
-            (
+            [
                 f"{key}->{value}"
                 for key, value in zip(["name", "cname", "axis", "pos", "kind"], temp)
-            )
+            ]
         )
 
     def __eq__(self, other: Any) -> bool:
@@ -2341,10 +2341,10 @@ def __repr__(self) -> str:
             )
         )
         return ",".join(
-            (
+            [
                 f"{key}->{value}"
                 for key, value in zip(["name", "cname", "dtype", "kind", "shape"], temp)
-            )
+            ]
         )
 
     def __eq__(self, other: Any) -> bool:

setup.cfg

@@ -106,6 +106,8 @@ ignore =
     B301,
     # single-letter variables
     PDF023
+    # "use 'pandas._testing' instead" in non-test code
+    PDF025
 exclude =
     doc/sphinxext/*.py,
     doc/build/*.py,