Revise SDK installation steps in README#1
Closed
angryfoxx wants to merge 1 commit intoUniversal-Commerce-Protocol:mainfrom
Closed
Revise SDK installation steps in README#1angryfoxx wants to merge 1 commit intoUniversal-Commerce-Protocol:mainfrom
angryfoxx wants to merge 1 commit intoUniversal-Commerce-Protocol:mainfrom
Conversation
0x8000-0000
approved these changes
Jan 14, 2026
Contributor
|
Thanks for you contribution. uv add ucp-sdk won't work if the pyproject.toml is not present. Can you add language around the dependency required to run uv-add? If not just remove that instruction. |
Author
no uv add works fine even if we dont specify in pyprojecttoml |
Contributor
|
I think you misunderstood my comment. Any user who is just following the instructions will run into an error if they are not in an initialized project directory. Either you add "uv init" command before running "uv add ucp-sdk" or you add some verbiage that the command should run from an initialized project directory. |
Updated installation instructions for the SDK.
Author
|
@aksbro-gpu get it, fixed |
pjordan
added a commit
to pjordan/python-sdk
that referenced
this pull request
Apr 16, 2026
Replace clone-based installation with PyPI instructions (pip/uv), add usage examples showing model imports and validation, add a model package reference table, and add PyPI/license badges. Closes Universal-Commerce-Protocol#1 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
pjordan
added a commit
to pjordan/python-sdk
that referenced
this pull request
Apr 16, 2026
Replace clone-based installation with PyPI instructions (pip/uv), add usage examples showing model imports and validation, add a model package reference table, and add PyPI/license badges. Closes Universal-Commerce-Protocol#1 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
4 tasks
pjordan
added a commit
to pjordan/python-sdk
that referenced
this pull request
Apr 16, 2026
Replace clone-based installation with PyPI instructions (pip/uv), add usage examples showing model imports and validation, add a model package reference table, and add PyPI/license badges. Closes Universal-Commerce-Protocol#1
pjordan
added a commit
that referenced
this pull request
May 4, 2026
* docs: improve README with PyPI install, usage examples, and badges Replace clone-based installation with PyPI instructions (pip/uv), add usage examples showing model imports and validation, add a model package reference table, and add PyPI/license badges. Closes #1 * docs: address review feedback on README examples Remove unused LineItem and Total imports from usage example. Move serialization inside try-except block and add missing Checkout import in validation example. * docs(readme): address PR review comments and fix prettier formatting * docs(readme): drop redundant Pydantic sentence in Usage section * docs(readme): replace Usage lead-in with example framing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated installation instructions for the SDK.