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

fix: llamaindex docs #1007

Merged
merged 2 commits into from
Dec 12, 2024
Merged

fix: llamaindex docs #1007

merged 2 commits into from
Dec 12, 2024

Conversation

abhishekpatil4
Copy link
Contributor

@abhishekpatil4 abhishekpatil4 commented Dec 12, 2024

Fixes ENG-2818


Important

Remove print(result) from Python code block in llamaindex.mdx.

  • Docs:
    • Remove print(result) from Python code block in llamaindex.mdx under "Execute the Agent" step.

This description was created by Ellipsis for 603329b. It will automatically update as commits are pushed.

Copy link

vercel bot commented Dec 12, 2024

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

Name Status Preview Comments Updated (UTC)
composio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 12, 2024 6:38pm

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to 603329b in 9 seconds

More details
  • Looked at 12 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. docs/framework/llamaindex.mdx:81
  • Draft comment:
    Consider keeping the print(result) statement to show users the output of the agent's action. This can help in understanding what the agent returns after execution.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The removal of the print statement might be intentional to keep the documentation clean, but it could be useful for users to see the output of the agent's action.

Workflow ID: wflow_zCu91hn2PEXG7w8A


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@Karthikeya-Meesala Karthikeya-Meesala merged commit 518695f into master Dec 12, 2024
12 checks passed
@Karthikeya-Meesala Karthikeya-Meesala deleted the fix/docs branch December 12, 2024 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants