Skip to content

ci: replace maturin upload step with uv publish#574

Merged
0x676e67 merged 1 commit into
mainfrom
ci
Apr 15, 2026
Merged

ci: replace maturin upload step with uv publish#574
0x676e67 merged 1 commit into
mainfrom
ci

Conversation

@0x676e67
Copy link
Copy Markdown
Owner

@0x676e67 0x676e67 commented Apr 15, 2026

close: #572

Summary by CodeRabbit

  • Chores
    • Updated the release automation pipeline with improved publishing infrastructure.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 31ea2526-c895-4491-8e6f-f1928f9b1501

📥 Commits

Reviewing files that changed from the base of the PR and between 2ed6f08 and 6907a8f.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The release workflow was updated to replace the deprecated PyO3/maturin-action with uv publish. A new step installs uv, followed by a dry-run publish step and a tag-gated step that publishes built wheels using the new tool with UV_PUBLISH_TOKEN.

Changes

Cohort / File(s) Summary
CI/CD Workflow Configuration
.github/workflows/ci.yml
Migrated from maturin-action to uv publish for package distribution. Removed job-level if condition, added uv installation step, replaced maturin upload with uv publish --dry-run (unconditional) and tag-gated uv publish wheels-*/* step. Token secret changed from MATURIN_PYPI_TOKEN to UV_PUBLISH_TOKEN.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 The maturin tool has had its day,
Now uv takes the wheels away!
With --dry-run before we ship,
Our packages make the gentler trip. 📦✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately and concisely describes the main change: replacing the maturin upload step with uv publish in the CI workflow.
Linked Issues check ✅ Passed The pull request successfully addresses issue #572 by replacing the deprecated maturin upload command with uv publish in the CI workflow.
Out of Scope Changes check ✅ Passed All changes are directly related to replacing maturin upload with uv publish in the CI workflow, with no out-of-scope modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@0x676e67 0x676e67 merged commit faca686 into main Apr 15, 2026
33 checks passed
@0x676e67 0x676e67 deleted the ci branch April 15, 2026 04:00
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.

maturin upload is deprecated

1 participant