Skip to content

chore: hanlp - avoid mutating documents and add tests#3206

Merged
anakin87 merged 1 commit intomainfrom
hanlp-test
Apr 22, 2026
Merged

chore: hanlp - avoid mutating documents and add tests#3206
anakin87 merged 1 commit intomainfrom
hanlp-test

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Apr 22, 2026

Related Issues

Proposed Changes:

  • avoid mutating documents in place
  • add unit tests, focusing on uncovered paths

How did you test it?

CI

Checklist

@github-actions
Copy link
Copy Markdown
Contributor

Coverage report (hanlp)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  integrations/hanlp/src/haystack_integrations/components/preprocessors/hanlp
  chinese_document_splitter.py
Project Total  

This report was generated by python-coverage-comment-action

@anakin87 anakin87 marked this pull request as ready for review April 22, 2026 11:05
@anakin87 anakin87 requested a review from a team as a code owner April 22, 2026 11:05
@anakin87 anakin87 requested review from davidsbatista and removed request for a team April 22, 2026 11:05
@anakin87 anakin87 self-assigned this Apr 22, 2026
Copy link
Copy Markdown
Contributor

@davidsbatista davidsbatista left a comment

Choose a reason for hiding this comment

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

Looks good!

deepset-ai/haystack#10956 -> please don't forget to update the PR so we know which integrations need to be fixed

@anakin87 anakin87 merged commit 3b0a2d7 into main Apr 22, 2026
15 checks passed
@anakin87 anakin87 deleted the hanlp-test branch April 22, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants