Skip to content

Conversation

@harupy
Copy link
Contributor

@harupy harupy commented Jan 23, 2026

Summary

  • Add uv sync as an alternative installation method in the Python SDK README

Test plan

  • Verify the README renders correctly on GitHub

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@harupy harupy requested a review from a team as a code owner January 23, 2026 15:42
Copilot AI review requested due to automatic review settings January 23, 2026 15:42
@harupy
Copy link
Contributor Author

harupy commented Jan 23, 2026

Unrelated to this PR, just out of curiosity, is there a reason the python SDK supports python < 3.10?

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds uv sync as an alternative installation method alongside pip install -e . in the Python SDK README. The change aims to provide developers with a modern package manager option for setting up the SDK.

Changes:

  • Added uv sync command as an alternative to pip install -e . in the Installation section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@friggeri friggeri added this pull request to the merge queue Jan 23, 2026
Merged via the queue into github:main with commit 0f4ed59 Jan 23, 2026
@friggeri
Copy link
Collaborator

Thanks for your contribution! And no, I've updated it to support all versions >=3.9

@harupy harupy deleted the docs/add-uv-sync-install-option branch January 24, 2026 01:47
@harupy
Copy link
Contributor Author

harupy commented Jan 24, 2026

@friggeri Thanks for the merge! Python 3.9 is already EOL-ed. Is there.a reason 3.9 needs to be supported?

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