Skip to content

adds uv test - #144

Closed
iitow wants to merge 3 commits into
mainfrom
test-uv
Closed

iitow wants to merge 3 commits into
mainfrom
test-uv

Conversation

@iitow

@iitow iitow commented Oct 21, 2026

Copy link
Copy Markdown

Summary

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Refactoring
  • Release / version bump

Checklist

  • Tests added/updated
  • Documentation updated
  • Tested E2E

SDK Parity

This is the Python SDK. If this change affects shared functionality, ensure the TypeScript SDK is updated as well.

Tip: Use the /sync-to-ts-sdk skill in Claude Code to automatically generate and create a TypeScript SDK PR from your changes.

  • Change is Python-specific (no TypeScript update needed)
  • TypeScript SDK PR created:

Copilot AI balanced review requested due to automatic review settings October 21, 2026 21:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

Static review found no blocking issues in this isolated workflow, though execution was not verified.

No open issues 🎉

What changed in this PR

Adds an isolated GitHub Actions workflow to test installing a pinned certifi release with uv, without changing SDK functionality.

Changes:

  • Runs manually or on pull requests that modify the workflow.
  • Installs uv 0.9.26, creates a Python 3.12 virtual environment, and installs certifi==2026.10.19 without caching.
  • Checks the installed version and confirms that certifi imports successfully.
File Description
.github/​workflows/​uv-download-test.yml Adds the uv installation test with read-only repository permissions and package verification.

Review effort: Balanced

@ryanrishi ryanrishi mentioned this pull request Oct 22, 2026
2 of 11 tasks
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