feat:added typescript sdk supports openai-sdk#40
Conversation
|
Thanks @Adarsh9977, this is a valuable contribution. To keep this reviewable and to avoid collisions with in-flight internal work, let's split this into three parts:
Also, pls ensure each PR has its relevant tests, no additional changes. Our restructure PR will go first and we'll ping you when it's time to rebase. Once you raise the two split PRs, we'll close this one in favor of them. Thank you! |
Thanks @shahtajkhalid for your review, I'll look into your mentioned points |
|
PR looks good overall, a few things to address first:
You can rebase from main now to get the new dir structure of the project that accounts for the new sdk folder. |
cool @yanny-sec, Thanks for this fast restructure. I'll make sure to cover all your points. |
…re capture registry
8f0ee04 to
abbc5eb
Compare
Summary
typescriptsdk supportsopenai-sdkas of now with proper documentation and examplesTest plan
From Repo root
For Specific Packages
Checklist