Closed Nisarg1112 closed 7 months ago
@Nisarg1112 Great work, thanks a lot! Will wait for the merged PR in langchain and merged this then! :)
Edit: Nevermind, merged! Will release this when langchain PR is merged.
Hey @Nisarg1112 I just checked your PR on langchain. Would recommend to remove all the trailing comma. Maybe your linter has a different setting, but to get things merged asap best practise is to keep the linter rules from the base repo 🙂
Hey, @fynnfluegge Sorry I missed these comments!
Sure it makes sense to remove trailing commas. Right now it's already approved actually.
Also, Thanks for merging my PR!