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

[Community]: requests_kwargs not being used in _fetch #28646

Merged
merged 3 commits into from
Dec 11, 2024
Merged

[Community]: requests_kwargs not being used in _fetch #28646

merged 3 commits into from
Dec 11, 2024

Conversation

keenborder786
Copy link
Contributor

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 10, 2024
@vercel Vercel
Copy link

vercel bot commented Dec 10, 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 Dec 11, 2024 11:43pm

@keenborder786 keenborder786 changed the title requests_kwargs not being used in _fetch [Community]: requests_kwargs not being used in _fetch Dec 10, 2024
@dosubot dosubot bot added community Related to langchain-community Ɑ: doc loader Related to document loader module (not documentation) labels Dec 10, 2024
efriis
efriis approved these changes Dec 11, 2024
View reviewed changes
Copy link
Member

@efriis efriis left a comment

Choose a reason for hiding this comment

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

made sure to merge them to avoid errors if they have the same key

@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Dec 11, 2024
@efriis efriis enabled auto-merge (squash) December 11, 2024 23:43
@efriis efriis self-assigned this Dec 11, 2024
@efriis efriis merged commit fa155a4 into langchain-ai:master Dec 11, 2024
19 checks passed
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. size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
PR Reviews
Status: Done
Milestone
No milestone
Development

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