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

docs: add LangFair as a provider #29390

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

docs: add LangFair as a provider #29390

wants to merge 13 commits into from

Conversation

virenbajaj
Copy link

@virenbajaj virenbajaj commented Jan 23, 2025 •
edited
Loading

Description:

  • Add docs/docs/providers/langfair.mdx
  • Register langfair in libs/packages.yml

Twitter handle: @LangFair

Tests and docs

  1. Integration tests not needed as this PR only adds a .mdx file to docs.

@vercel Vercel
Copy link

vercel bot commented Jan 23, 2025 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2025 2:36pm

dylanbouchard
dylanbouchard reviewed Jan 24, 2025
View reviewed changes
libs/packages.yml Outdated Show resolved Hide resolved
@dylanbouchard
Copy link

dylanbouchard commented Jan 28, 2025 •
edited
Loading

@ccurme, @efriis the Vercel check is the only one that is failing, but I am unable to view the reason for this. When I click on details, it fails to load. Can you please advise?

image

efriis
efriis reviewed Jan 29, 2025
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.

hey there! you should have either an ipynb or an mdx file. It looks like the contents might be slightly different - did you have one that you wanted to keep over the other?

@efriis
Copy link
Member

efriis commented Jan 29, 2025

it looks like vercel removed the ability for OSS contributors to view the build logs recently :( - will check back in once the ipynb/mdx file thing is addressed!

@virenbajaj virenbajaj marked this pull request as ready for review January 29, 2025 14:38
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Jan 29, 2025
@virenbajaj
Copy link
Author

@efriis just removed .ipynb and the checks pass now. Thanks for the tip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@efriis efriis efriis left review comments

@dylanbouchard dylanbouchard dylanbouchard approved these changes

Assignees
No one assigned
Labels
🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder size:L This PR changes 100-499 lines, ignoring generated files.
Projects
PR Reviews
Status: Triage
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@virenbajaj @dylanbouchard @efriis @ccurme