Pre-commit-check

Author: parthkandharkar

pandas/io/formats/css.py

@@ -19,13 +19,8 @@
         Iterator,
     )
 
-def _normalize_number_format_value(value: str) -> str:
-    """
-    Lowercase number-format value except for text inside double quotes.
 
-    This preserves case for string literals (e.g. "M") while still
-    normalizing things like [Red] -> [red].
-    """
+def _normalize_number_format_value(value: str) -> str:
     value = value.strip()
     out: list[str] = []
     in_string = False
@@ -34,11 +29,10 @@ def _normalize_number_format_value(value: str) -> str:
         if ch == '"':
             out.append(ch)
             in_string = not in_string
+        elif in_string:
+            out.append(ch)  # preserve case inside string literals
         else:
-            if in_string:
-                out.append(ch)       # preserve case inside string literals
-            else:
-                out.append(ch.lower())  # normalize outside literals
+            out.append(ch.lower())  # normalize outside literals
     return "".join(out)
 
 

pandas/tests/io/excel/test_style.py

@@ -96,7 +96,7 @@ def test_styler_to_excel_unstyled(engine, tmp_excel):
     ("font-style: italic;", ["font", "i"], True),
     ("text-decoration: underline;", ["font", "u"], "single"),
     ("number-format: $??,???.00;", ["number_format"], "$??,???.00"),
-    ("number-format: #,,"'"M";', ["number_format"], '#,,"M"'),
+    ('number-format: #,,"M";', ["number_format"], '#,,"M"'),
     ("text-align: left;", ["alignment", "horizontal"], "left"),
     (
         "vertical-align: bottom;",

pandas/tests/io/formats/test_css.py

@@ -287,6 +287,7 @@ def test_css_relative_font_size(size, relative_to, resolved):
         inherited = {"font-size": relative_to}
     assert_resolves(f"font-size: {size}", {"font-size": resolved}, inherited=inherited)
 
+
 def test_css_atomize_preserves_number_format_case():
     resolver = CSSResolver()
     declarations = [("NUMBER-FORMAT", '#,,"M"')]
@@ -296,6 +297,7 @@ def test_css_atomize_preserves_number_format_case():
     assert "number-format" in props
     assert props["number-format"] == '#,,"M"'
 
+
 def test_css_parse_preserves_number_format_case():
     resolver = CSSResolver()