Repository navigation
ci: publish Rust crate releases - #19
Merged
Merged
Conversation
qartik
requested review from
chesterwringe and
qfreewilll
and
a balanced review from Copilot
September 24, 2026 13:31
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The documented installation command does not create the dependency alias required by the Rust usage example.
Get a fresh assessment by requesting another Copilot review.
Review effort: Balanced
Findings: 1
Open (1)
What changed in this PR
Publishes the Rust crate to crates.io while preserving PyPI releases.
Changes:
- Adds crates.io publishing and package verification.
- Adds crate discovery metadata.
- Documents Rust installation and dependency aliasing.
| File | Description |
|---|---|
.github/workflows/release.yml |
Publishes releases to crates.io. |
Cargo.toml |
Adds crates.io metadata. |
README.md |
Documents Rust installation and usage. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
qartik
marked this pull request as ready for review
September 24, 2026 13:54
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
qfreewilll
approved these changes
Sep 24, 2026
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
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.
Summary
qir-formatterto crates.io when a GitHub Release is createdValidation
cargo publish --dry-run --locked --allow-dirtyuv run src/scripts/ci_checks.sh: Clippy, formatting, ty, 17 Rust tests, 1 doctest, and 38 Python testscp310-abi3wheel and sdist buildsabi3audit --strictverifies the extension's Python 3.10 stable-ABI baselinetwine check --strictanduv publish --dry-runaccept the wheel and sdist upload setThe release-only wheel matrix cannot run on this pull request. Its publish job depends on every Linux, macOS, Windows, and sdist build-and-install test, so PyPI publishing is blocked if any target fails rather than uploading a partial release.