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.

Sign up for GitHub

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

Jump to bottom

fix: text splitter with tiny documents (fixes #193) #196

Merged
merged 1 commit into from Mar 22, 2024
Merged

fix: text splitter with tiny documents (fixes #193) #196

merged 1 commit into from Mar 22, 2024

Conversation

sinedied
Copy link
Contributor

Purpose

Tested on a very small doc, works fine

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

manekinekko
manekinekko approved these changes Mar 22, 2024
View reviewed changes
Copy link
Contributor

@manekinekko manekinekko left a comment

Choose a reason for hiding this comment

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

LGTM

@manekinekko manekinekko merged commit 2e808c7 into main Mar 22, 2024
6 checks passed
@manekinekko manekinekko deleted the fix-indexer branch March 22, 2024 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manekinekko manekinekko manekinekko approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sinedied @manekinekko