-
Notifications
You must be signed in to change notification settings - Fork 310
docs(community): update docstrings for reference docs #1237 #1255
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
base: main
Are you sure you want to change the base?
docs(community): update docstrings for reference docs #1237 #1255
Conversation
Hey @mdrxy! here is the PR for the community pkg for new ref docs. Working on these docstring updates raised some questions about reference docs patterns: 1. Documenting private method details in class docstrings: 2. Example detail level in reference docs: 3. Package organization in docs: PS: Asking this not to give you more trouble, just to say I can help with some of those if they're not already planned |
Appreciate the checking in and feel free to reach out on Slack if quicker |
on behalf of the associated account. | ||
See https://python.langchain.com/docs/security for more information. | ||
See [Security Best Practices](https://python.langchain.com/docs/security) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will need to update this (and all other links) once the security page is ported (cc @eyurtsev)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added TODO comments in calendar/toolkit.py
and
gmail/toolkit.py
to update these links once the security page is ported.
libs/community/langchain_google_community/sheets/create_spreadsheet_tool.py
Outdated
Show resolved
Hide resolved
@mdrxy I'll keep an eye out for the .md patch from ChatOpenAI and take a look at mkdocs.yml to help with the docs for the Google packages Google Sheets examples in this PR:
Thanks again for the review and clarifications! |
Description
Updated docstrings in
langchain-google-community
to be in accordance with the new reference docs site.Relevant issues
Fixes #1237
Type
📖 Documentation