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

Improvement[Community] Improve Document Loaders and Splitters #27568

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Improvement[Community] Improve Document Loaders and Splitters #27568

merged 2 commits into from
Oct 24, 2024

Conversation

ZhangShenao
Copy link
Contributor

  • Fix word spelling error
  • Add static method decorator
  • Fix language splitter

@vercel Vercel
Copy link

vercel bot commented Oct 23, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Oct 24, 2024 9:38pm

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. community Related to langchain-community Ɑ: doc loader Related to document loader module (not documentation) Ɑ: text splitters Related to text splitters package 🤖:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs labels Oct 23, 2024
@ZhangShenao
Copy link
Contributor Author

@efriis @eyurtsev @ccurme Please review this pr

@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Oct 24, 2024
@efriis efriis enabled auto-merge (squash) October 24, 2024 21:38
@efriis efriis self-assigned this Oct 24, 2024
@efriis efriis merged commit 455ab7d into langchain-ai:master Oct 24, 2024
53 checks passed
@ZhangShenao ZhangShenao deleted the improvement-document-loader branch October 25, 2024 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@efriis efriis efriis approved these changes

Assignees

@efriis efriis

Labels
community Related to langchain-community Ɑ: doc loader Related to document loader module (not documentation) lgtm PR looks good. Use to confirm that a PR is ready for merging. 🤖:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs size:XS This PR changes 0-9 lines, ignoring generated files. Ɑ: text splitters Related to text splitters package
Projects
PR Reviews
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ZhangShenao @efriis