Skip to content

Use uv publish for release workflow - #1

Merged
mgaitan merged 14 commits into
mainfrom
feature/python-api-complete
Dec 18, 2025
Merged

Use uv publish for release workflow#1
mgaitan merged 14 commits into
mainfrom
feature/python-api-complete

Conversation

@mgaitan

@mgaitan mgaitan commented Dec 18, 2025

Copy link
Copy Markdown
Owner

Summary

  • publish release wheels using uv publish with trusted publishing
  • keep wheels attached as GitHub release assets
  • set OIDC permissions for release job

Testing

  • not run (CI workflow change only)

Comment thread .github/workflows/python-wheels.yml Fixed
Comment thread .github/workflows/python-wheels.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/python-wheels.yml Fixed
Comment thread .github/workflows/python-wheels.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/python-wheels.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/test.yml Fixed
Comment thread .github/workflows/test.yml Fixed
- uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5
with:
path: wheelhouse
- uses: astral-sh/setup-uv@681c641aba71e4a1c380be3ab5e12ad51f415867 # v7

Check failure

Code scanning / zizmor

runtime artifacts potentially vulnerable to a cache poisoning attack Error

runtime artifacts potentially vulnerable to a cache poisoning attack
Comment on lines +58 to +59
- name: Checkout
uses: actions/checkout@8ade135a41bc03ea155e62e844d188df1ea18608 # v4.1.0

Check warning

Code scanning / zizmor

credential persistence through GitHub Actions artifacts: does not set persist-credentials: false Warning test

credential persistence through GitHub Actions artifacts: does not set persist-credentials: false
@mgaitan
mgaitan merged commit 1f71bb5 into main Dec 18, 2025
4 of 5 checks passed
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