CI: prevent xgboost build-script panic in Rust build & test workflow - #8
Conversation
Co-authored-by: singjc <32938975+singjc@users.noreply.github.com>
|
@copilot the GHA is still failing? https://github.com/singjc/redeem/actions/runs/34393258999/job/102646318068?pr=8 |
Co-authored-by: singjc <32938975+singjc@users.noreply.github.com>
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
You’re right — it was still failing in the same xgboost build-script path. I pushed |
The
Rust build & testGitHub Actions job failed duringcargo build --features xgboostwhenxgboost_lib-syspanicked on a missingtarget/debug/depspath. This PR hardens the workflow by creating that path before the xgboost-enabled build step.Root cause addressed
xgboost_lib-sysbuild script canonicalizestarget/debug/depsand panics if it is absent on a clean runner.Workflow change
.github/workflows/rust.ymlimmediately before the xgboost build step.Resulting execution order