8000 Fix ruff errors · kosiew/datafusion-python@2da8da9 · GitHub
[go: up one dir, main page]

Skip to content

Commit 2da8da9

Browse files
committed
Fix ruff errors
1 parent f5bec5b commit 2da8da9

File tree

2 files changed

+19
-10
lines changed

2 files changed

+19
-10
lines changed

python/datafusion/html_formatter.py

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -647,18 +647,28 @@ def configure_formatter(**kwargs: Any) -> None:
647647
"""
648648
# Valid parameters accepted by DataFrameHtmlFormatter
649649
valid_params = {
650-
"max_cell_length", "max_width", "max_height", "max_memory_bytes",
651-
"min_rows_display", "repr_rows", "enable_cell_expansion", "custom_css",
652-
"show_truncation_message", "style_provider", "use_shared_styles"
650+
"max_cell_length",
651+
"max_width",
652+
"max_height",
653+
"max_memory_bytes",
654+
"min_rows_display",
655+
"repr_rows",
656+
"enable_cell_expansion",
657+
"custom_css",
658+
"show_truncation_message",
659+
"style_provider",
660+
"use_shared_styles",
653661
}
654-
662+
655663
# Check for invalid parameters
656664
invalid_params = set(kwargs) - valid_params
657665
if invalid_params:
658-
msg = f"Invalid formatter parameters: {', '.join(invalid_params)}. " \
659-
f"Valid parameters are: {', '.join(valid_params)}"
666+
msg = (
667+
f"Invalid formatter parameters: {', '.join(invalid_params)}. "
668+
f"Valid parameters are: {', '.join(valid_params)}"
669+
)
660670
raise ValueError(msg)
661-
671+
662672
# Create and set formatter with validated parameters
663673
set_formatter(DataFrameHtmlFormatter(**kwargs))
664674

python/tests/test_dataframe.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1055,11 +1055,11 @@ def test_configure_formatter_invalid_params(clean_formatter_state):
10551055
"""Test that configure_formatter rejects invalid parameters."""
10561056
with pytest.raises(ValueError, match="Invalid formatter parameters"):
10571057
configure_formatter(invalid_param=123)
1058-
1058+
10591059
# Test with multiple parameters, one valid and one invalid
10601060
with pytest.raises(ValueError, match="Invalid formatter parameters"):
10611061
configure_formatter(max_width=500, not_a_real_param="test")
1062-
1062+
10631063
# Test with multiple invalid parameters
10641064
with pytest.raises(ValueError, match="Invalid formatter parameters"):
10651065
configure_formatter(fake_param1="test", fake_param2=456)
@@ -1794,4 +1794,3 @@ def test_html_formatter_manual_format_html(clean_formatter_state):
17941794

17951795
assert "<style>" in local_html_1
17961796
assert "<style>" in local_html_2
1797-

0 commit comments

Comments
 (0)
0