Skip to content

fix(community): export openapi util #8037

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
Merged
merged 1 commit into from
Apr 28, 2025
Merged

fix(community): export openapi util #8037

merged 1 commit into from
Apr 28, 2025

Conversation

schipiga
Copy link
Contributor

@schipiga schipiga commented Apr 17, 2025 •
edited
Loading

Make OpenAPISpec class available outside of package in order to use with function convertOpenAPISpecToOpenAIFunctions from #7955

@vercel Vercel
Copy link

vercel bot commented Apr 17, 2025 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Apr 28, 2025 11:04am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Apr 28, 2025 11:04am

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 17, 2025
@schipiga schipiga changed the title Export openapi util fix(community): export openapi util Apr 17, 2025
@dosubot dosubot bot added the auto:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs label Apr 17, 2025
@schipiga
Copy link
Contributor Author

schipiga commented Apr 17, 2025 •
edited
Loading

Hi @jacoblee93 , sorry but with my previous pr #7955 I completely missed to make argument class exportable also. This PR fixes the issue.

@schipiga schipiga force-pushed the fix/export-openapi-util branch from 0ed9598 to add7ec2 Compare April 17, 2025 05:48
@benjamincburns benjamincburns force-pushed the fix/export-openapi-util branch from add7ec2 to 477d4e7 Compare April 23, 2025 12:53
@benjamincburns benjamincburns force-pushed the fix/export-openapi-util branch from cf7d5ad to 5390d91 Compare April 28, 2025 10:51
@benjamincburns benjamincburns merged commit af108cc into langchain-ai:main Apr 28, 2025
30 checks passed
@benjamincburns benjamincburns mentioned this pull request Apr 30, 2025
Merged
@benjamincburns
Copy link
Collaborator

Unfortunately this change was reverted. The build scripts were overwriting the changes to the package.json file automatically. I'll need to follow up with this later update the build config.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
auto:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@schipiga @benjamincburns