Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: Update API, naming, and tests. Move data model to docling-core #107

Merged
merged 16 commits into from
Mar 14, 2025

Conversation

cau-git
Copy link
Contributor

@cau-git cau-git commented Mar 10, 2025

Copy link

mergify bot commented Mar 10, 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)(?:\(.+\))?(!)?:

cau-git added 5 commits March 11, 2025 13:12
Signed-off-by: Christoph Auer <[email protected]>
Signed-off-by: Christoph Auer <[email protected]>
Signed-off-by: Christoph Auer <[email protected]>
Signed-off-by: Christoph Auer <[email protected]>
@cau-git cau-git changed the title feat: Update API, naming, and tests. Split data models to be independent. feat!: Update API, naming, and tests. Split data models to be independent. Mar 12, 2025
@cau-git
Copy link
Contributor Author

cau-git commented Mar 12, 2025

TODO

  • Align member names on C++ code with python API changes (ordering, left_to_right)
  • Update README
  • Deprecate v1 backend

cau-git added 2 commits March 12, 2025 16:34
Signed-off-by: Christoph Auer <[email protected]>
Signed-off-by: Christoph Auer <[email protected]>
@cau-git cau-git force-pushed the cau/api-move-to-docling-core branch from 06412f1 to 04096f2 Compare March 12, 2025 18:33
Signed-off-by: Christoph Auer <[email protected]>
@cau-git cau-git changed the title feat!: Update API, naming, and tests. Split data models to be independent. feat!: Update API, naming, and tests. Move data model to docling-core Mar 13, 2025
@cau-git cau-git marked this pull request as ready for review March 13, 2025 14:47
@cau-git cau-git requested a review from PeterStaar-IBM March 13, 2025 18:02
PeterStaar-IBM
PeterStaar-IBM previously approved these changes Mar 14, 2025
Copy link
Contributor

@PeterStaar-IBM PeterStaar-IBM left a comment

Choose a reason for hiding this comment

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

Wonderful!

Signed-off-by: Michele Dolfi <[email protected]>
dolfim-ibm
dolfim-ibm previously approved these changes Mar 14, 2025
cau-git added 2 commits March 14, 2025 10:58
Signed-off-by: Christoph Auer <[email protected]>
@cau-git cau-git dismissed stale reviews from dolfim-ibm and PeterStaar-IBM via e450ecc March 14, 2025 09:58
@cau-git cau-git merged commit ca7d584 into main Mar 14, 2025
57 checks passed
@cau-git cau-git deleted the cau/api-move-to-docling-core branch March 14, 2025 12:00
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