Skip to content

Conversation

amitalokbera
Copy link

@amitalokbera amitalokbera commented Jun 22, 2025

Important

Update installation instructions and add available extras for traceloop-sdk in openllmetry/getting-started-python.mdx.

  • Installation Instructions:
    • Updated to include installation of all optional dependencies using traceloop-sdk[all].
    • Added instructions for installing specific extras using a comma-separated list.
  • Available Extras:
    • Listed available extras such as mistralai, openai, ollama, langchain, and others with their corresponding opentelemetry-instrumentation packages.

This description was created by Ellipsis for ace19f8. You can customize this summary. It will automatically update as commits are pushed.

@amitalokbera amitalokbera marked this pull request as ready for review June 22, 2025 14:19
Copy link

@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.

Caution

Changes requested ❌

Reviewed everything up to ace19f8 in 1 minute and 28 seconds. Click for details.
  • Reviewed 69 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. openllmetry/getting-started-python.mdx:39
  • Draft comment:
    Consider sorting the 'Available Extras' list alphabetically for better usability.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
2. openllmetry/getting-started-python.mdx:26
  • Draft comment:
    Typo: "comma-seperated" should be "comma-separated".
  • Reason this comment was not posted:
    Marked as duplicate.

Workflow ID: wflow_yctD9ImzVzdvRnOH

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
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.

1 participant