Skip to content

fix(service): Fixes for Docling ConfidenceReport parsing in ConversionResult#3662

Merged
PeterStaar-IBM merged 2 commits into
mainfrom
cau/confidence-report-from-service-parsing
Jun 19, 2026
Merged

fix(service): Fixes for Docling ConfidenceReport parsing in ConversionResult#3662
PeterStaar-IBM merged 2 commits into
mainfrom
cau/confidence-report-from-service-parsing

Conversation

@cau-git

@cau-git cau-git commented Jun 19, 2026

Copy link
Copy Markdown
Member

Checklist:

  • Documentation has been updated, if necessary.
  • Examples have been added, if necessary.
  • Tests have been added, if necessary.

Signed-off-by: Christoph Auer <cau@zurich.ibm.com>
@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

DCO Check Passed

Thanks @cau-git, all your commits are properly signed off. 🎉

@mergify

mergify Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

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)(?:\(.+\))?(!)?:

@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
docling/datamodel/base_models.py 85.71% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

Signed-off-by: Christoph Auer <cau@zurich.ibm.com>

@dolfim-ibm dolfim-ibm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@PeterStaar-IBM PeterStaar-IBM merged commit 898dd73 into main Jun 19, 2026
26 checks passed
@PeterStaar-IBM PeterStaar-IBM deleted the cau/confidence-report-from-service-parsing branch June 19, 2026 12:57
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