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

fix: Fixed docx import with headers that are also lists #842

Merged
merged 3 commits into from
Jan 31, 2025

Conversation

maxmnemonic
Copy link
Contributor

@maxmnemonic maxmnemonic commented Jan 30, 2025

Now such headers are converted as appropriate headers and subheaders, unit test included

Issue resolved by this Pull Request:
Resolves #612
Resolves #795
Resolves #738

Checklist:

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

… headers and subheaders, unit test included

Signed-off-by: Maksym Lysak <[email protected]>
@maxmnemonic maxmnemonic added the docx issue related to docx backend label Jan 30, 2025
@maxmnemonic maxmnemonic self-assigned this Jan 30, 2025
Copy link

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

🟢 Require two reviewer for test updates

Wonderful, this rule succeeded.

When test data is updated, we require two reviewers

  • #approved-reviews-by >= 2

dolfim-ibm
dolfim-ibm previously approved these changes Jan 30, 2025
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.

nice!

ceberam
ceberam previously approved these changes Jan 31, 2025
Copy link
Contributor

@ceberam ceberam left a comment

Choose a reason for hiding this comment

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

Just added minor suggestions for readability (do not declare variables that are never used).

docling/backend/msword_backend.py Outdated Show resolved Hide resolved
docling/backend/msword_backend.py Outdated Show resolved Hide resolved
Co-authored-by: Cesar Berrospi Ramis <[email protected]>
Signed-off-by: Maxim Lysak <[email protected]>
@maxmnemonic maxmnemonic dismissed stale reviews from ceberam and dolfim-ibm via 53f6d95 January 31, 2025 09:20
Co-authored-by: Cesar Berrospi Ramis <[email protected]>
Signed-off-by: Maxim Lysak <[email protected]>
@maxmnemonic
Copy link
Contributor Author

Just added minor suggestions for readability (do not declare variables that are never used).

nice! approved those commits

Copy link
Contributor

@ceberam ceberam left a comment

Choose a reason for hiding this comment

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

Great!

@maxmnemonic maxmnemonic merged commit 2c037ae into main Jan 31, 2025
9 checks passed
@maxmnemonic maxmnemonic deleted the dev/docx_list_headers branch January 31, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docx issue related to docx backend
Projects
None yet
4 participants