Skip to content

Conversation

@hassiebp
Copy link
Contributor

@hassiebp hassiebp commented Jul 17, 2025

Important

Update documentation and improve API organization for v3 release.

  • Documentation:
    • Update README.md to use CDN-hosted logo and update version references from v2 to v3.
    • Simplify documentation links in README.md to centralize v3 SDK documentation.
  • API Improvements:
    • Remove OpenTelemetry span attributes from public API in langfuse/__init__.py.
    • Add LangfuseSpan, LangfuseGeneration, LangfuseEvent to public API in langfuse/__init__.py.
    • Introduce __all__ in langfuse/__init__.py for explicit public interface definition.

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


Disclaimer: Experimental PR review

Greptile Summary

This PR makes two main types of changes:

  1. Documentation updates in README.md:

    • Updates the banner image URL to use a CDN-hosted white logo
    • Updates version references from v2 to v3
    • Simplifies documentation by centralizing links to v3 SDK documentation
  2. API improvements in langfuse/__init__.py:

    • Removes OpenTelemetry span attributes from public API
    • Adds three new classes to public API: LangfuseSpan, LangfuseGeneration, LangfuseEvent
    • Introduces __all__ declaration for explicit public interface definition

These changes reflect a professional effort to improve documentation organization and API cleanliness as part of the v3 release.

Confidence score: 4/5

  1. This PR is safe to merge as it primarily contains documentation and API organization improvements
  2. High confidence due to the non-breaking nature of changes and clear improvement in API organization
  3. Files needing attention:
    • langfuse/__init__.py - Verify that removed OpenTelemetry attributes won't affect existing users

2 files reviewed, no comments
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Reviewing changes made in this pull request

@hassiebp hassiebp merged commit d0806f3 into main Jul 17, 2025
8 checks passed
@hassiebp hassiebp deleted the update-pydoc branch July 17, 2025 14:56
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