Skip to content

Commit d4eaf94

Browse files
Smeet23claude
andcommitted
refactor: move Excel markdown serializer to docling-core
Remove `docling/utils/markdown.py` and update imports to use `docling_core.transforms.serializer.markdown_excel` as requested in code review on #3255. The serializer now lives in docling-core#587. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 3dce367 commit d4eaf94

File tree

2 files changed

+6
-68
lines changed

2 files changed

+6
-68
lines changed

docling/utils/markdown.py

Lines changed: 0 additions & 66 deletions
This file was deleted.

tests/test_backend_msexcel.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,9 @@ def documents() -> list[tuple[Path, DoclingDocument]]:
5959

6060

6161
def test_e2e_excel_conversions(documents) -> None:
62-
from docling.utils.markdown import MsExcelMarkdownDocSerializer
62+
from docling_core.transforms.serializer.markdown_excel import (
63+
MsExcelMarkdownDocSerializer,
64+
)
6365

6466
for gt_path, doc in documents:
6567
pred_md: str = MsExcelMarkdownDocSerializer(doc=doc).serialize().text
@@ -282,7 +284,9 @@ def test_sheet_names_as_headings(documents) -> None:
282284
injected into the document model by the backend.
283285
Sheet4 in xlsx_01 is empty and intentionally has no heading.
284286
"""
285-
from docling.utils.markdown import MsExcelMarkdownDocSerializer
287+
from docling_core.transforms.serializer.markdown_excel import (
288+
MsExcelMarkdownDocSerializer,
289+
)
286290

287291
doc = next(item for path, item in documents if path.stem == "xlsx_01")
288292

0 commit comments

Comments
 (0)