Skip to content

add cursor rules#98

Merged
Jose-Sabater merged 1 commit intomainfrom
add-ai-rules
Jun 30, 2025
Merged

add cursor rules#98
Jose-Sabater merged 1 commit intomainfrom
add-ai-rules

Conversation

@Jose-Sabater
Copy link
Member

No description provided.

@Jose-Sabater Jose-Sabater merged commit bad4c0f into main Jun 30, 2025
1 check passed
@Jose-Sabater Jose-Sabater deleted the add-ai-rules branch June 30, 2025 13:48
Copy link

@agentobot agentobot bot left a comment

Choose a reason for hiding this comment

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

Opper Python Docs PR Review

Highlights

  • Comprehensive end-to-end SDK documentation covering schemas, tracing, metrics, and knowledge base.
  • 📝 Clear code examples and helpful conceptual explanations.
  • 🔗 Minor README intro improvement.

Suggestions (non-blocking)

Area Observation Recommendation
Install Instructions A quick-start install command (pip install opper-python) is missing. Add an Installation snippet near the top so new users can start in one copy-paste.
Python Version Support The docs do not mention the minimum supported Python version. Specify the minimum version (e.g., "Requires Python >= 3.8") to set expectations.
Async Tracing Example The child-span example mixes synchronous style in an async context; await is omitted. Provide an explicit async def example with await client.spans.start_child(...) for clarity.
Error Handling Examples omit retries or structured exception handling. Add a short note (e.g., try/except + retries=3 argument) to illustrate best practice.
Link Context The link to the model list works, but lacks descriptive context. Consider adding a brief inline description (e.g., "See full model list →").

Overall Impression

Great work! The documentation is thorough and elevates the SDK’s usability. The points above are minor suggestions and do not block merge.

Approved ✅

— agentobot[bot]

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