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.

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

docs: add Composio integration details #29238

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

Conversation

abhishekpatil4
Copy link

  • Description: This PR adds documentation for the Composio toolset usage with LangChain

Copy link

vercel bot commented Jan 15, 2025

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 16, 2025 7:07am

@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 15, 2025
@efriis
Copy link
Member

efriis commented Jan 15, 2025

hey there! Here's the guide on registering an integration package like this: https://python.langchain.com/docs/contributing/how_to/integrations/publish/#adding-documentation-to-the-langchain-monorepo

If you could fix it this week that would be great, as well as get it to pass CI

@efriis efriis self-assigned this Jan 15, 2025
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jan 16, 2025
@abhishekpatil4
Copy link
Author

Hi @efriis . I followed the guide shared by you earlier & added the docs to Provider Page and Tools Page. However, I'm unsure about registering the package in libs/packages.yml because the package composio-langchain is developed and maintained by Composio, and the template for packages.yml requires specifying the path of the package.

Additionally, I'm encountering a lint error even before my changes were made in the master branch:

poetry run ruff check docs cookbook
All checks passed!
poetry run ruff format docs cookbook cookbook --diff
1151 files already formatted
poetry run ruff check --select I docs cookbook
All checks passed!
git --no-pager grep 'from langchain import' docs cookbook | grep -vE 'from langchain import (hub)' && echo "Error: no importing langchain from root in docs, except for hub" && exit 1 || exit 0
git --no-pager grep 'api.python.langchain.com' -- docs/docs ':!docs/docs/additional_resources/arxiv_references.mdx' ':!docs/docs/integrations/document_loaders/sitemap.ipynb' || exit 0 && \
        echo "Error: you should link python.langchain.com/api_reference, not api.python.langchain.com in the docs" && \
        exit 1

Could you please provide guidance on how to proceed with the registration in packages.yml & and fixing the lint error?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder size:XL This PR changes 500-999 lines, ignoring generated files. size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

2 participants