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: Fix typo in _action_agent docs section #27698

Merged
merged 3 commits into from
Oct 29, 2024
Merged

Conversation

JminJ
Copy link
Contributor

@JminJ JminJ commented Oct 29, 2024

PR Title: docs: Fix typo in _action_agent function docs section

Description: In line 1185, _action_agent function's docs, changing ".agent" to "self.agent".

Issue: N/A

Dependencies: None

Copy link

vercel bot commented Oct 29, 2024

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 Oct 29, 2024 2:16pm

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. Ɑ: agent Related to agents module 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Oct 29, 2024
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Oct 29, 2024
@eyurtsev
Copy link
Collaborator

It's somewhat unusual to document attributes with the self namespace.

@eyurtsev eyurtsev enabled auto-merge (squash) October 29, 2024 14:08
@eyurtsev eyurtsev self-assigned this Oct 29, 2024
@eyurtsev eyurtsev merged commit 0a465b8 into langchain-ai:master Oct 29, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ɑ: agent Related to agents module 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder lgtm PR looks good. Use to confirm that a PR is ready for merging. size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants