Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughAdded Depot PR runner activation canary comments to two READMEs, two build scripts, two SDK READMEs, and the website template. No runtime, rendered output, or documented behavior changed. ChangesDepot runner activation canaries
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This canary changes comments and documentation-oriented content without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review. Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
|
Canary complete: all five Depot attempt-2 lanes passed at exact SHA 631df71, then all five gate-absent hosted attempt-3 lanes passed with exact job-name graph parity. The three bounded-exception variables were removed before rollback. This comment-only canary is intentionally closed without merge; durable evidence is in #1341. |
Purpose
Controlled, non-merge activation canary for the time-bounded Depot PR risk exception. The source changes are comments only, but their owned paths deliberately select Quality, Website/UI, Linux, macOS, Windows, all runtime backend rows, Rust tests, and Rust/Kotlin/Swift SDK rows.
Sequence
DEPOT_PR_RUNNERS_ENABLED=true,DEPOT_PR_APPROVED_REF=refs/pull/<this>/merge, andDEPOT_PR_APPROVED_SHA=<this exact head>.Known exceptions remain hosted/self-hosted: planning and stable summaries, credential-bearing smoke consumers, and the approved CUDA GPU hardware smoke. This canary knowingly accepts the repository-wide Depot cache risk documented in
ci/DEPOT_PR_RISK_EXCEPTION.md.Summary by CodeRabbit