ci: fix missing toolchain input in release publish job#18
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughPublish job in the GitHub Actions workflow was modified: it now runs on Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
The publish job was on ubuntu-latest with dtolnay/rust-toolchain (which failed due to missing toolchain input). Switch to cachekit-lean runners with rustup directly — same as CI. Drop dtolnay action and Swatinem/rust-cache (unnecessary on persistent runners).
7acec7c to
02d02f9
Compare
Summary
dtolnay/rust-toolchainaction in the publish job was SHA-pinned but missing the requiredtoolchain: stableinput'toolchain' is a required inputFix
Add
with: toolchain: stableto the step.After merging, re-run the failed publish job or manually
cargo publish.Summary by CodeRabbit