ci: trigger package portability for Rust build configuration - #678
Conversation
Signed-off-by: Alex Fournier <afournier@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 6 remain after this review. WalkthroughThe package portability workflow now triggers when ChangesPackage portability workflow
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to Rust and Cargo configuration changes will now run package portability checks for pull requests and pushes to main. No merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
A rabbit checks the Rusty gate Comment |
|
For reference, Relay includes Relay does not currently have a |
What
Run Package Portability when
rust-toolchain.tomlor.cargo/config.tomlchanges.Why
Both files affect the Linux wheel build, but neither currently triggers the workflow. A Rust toolchain or target flag change can therefore reach
mainwithout rebuilding and smoke-testing the wheel.How tested
go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.7 .github/workflows/package-portability.ymlgit diff --checkNotes for reviewers
This only changes when Package Portability runs. The wheel build, test commands, package configuration, release workflow, artifacts, and publishing behavior are unchanged.
Summary by CodeRabbit