Skip to content

Conversation

@maxmnemonic
Copy link
Contributor

@maxmnemonic maxmnemonic commented Oct 7, 2025

Moving log of deleting elements from info level to debug.
This is due to support of rich tables, while constructing them backends (docx, html) manipulate docling document tree, and sometimes need to delete elements, this creates excessive logging.

Note from the user: docling-project/docling#2360 (comment)

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

DCO Check Passed

Thanks @maxmnemonic, all your commits are properly signed off. 🎉

@dosubot
Copy link

dosubot bot commented Oct 7, 2025

Related Documentation

Checked 2 published document(s). No updates required.

You have 5 draft document(s). Publish docs to keep them always up-to-date

How did I do? Any feedback?  Join Discord

@maxmnemonic maxmnemonic requested review from cau-git and vagenas October 7, 2025 08:13
@maxmnemonic maxmnemonic self-assigned this Oct 7, 2025
@mergify
Copy link

mergify bot commented Oct 7, 2025

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

Copy link
Contributor

@dolfim-ibm dolfim-ibm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@maxmnemonic maxmnemonic merged commit 2b519f2 into main Oct 7, 2025
12 checks passed
@maxmnemonic maxmnemonic deleted the dev/silence_delete_items branch October 7, 2025 12:37
@dosubot
Copy link

dosubot bot commented Oct 7, 2025

Documentation Updates

Checked 2 published document(s). No updates required.

You have 5 draft document(s). Publish docs to keep them always up-to-date

How did I do? Any feedback?  Join Discord

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants