Repository navigation
build: add Bazelisk and drift checks to pre-push - #453
Conversation
Require bazelisk in local preflight and pre-push-fast; run cargo-bazel-drift-check early. Add make bazel-test and surface docs/development/bazel.md from AGENTS/CONTRIBUTING/onboarding. Closes #447 Co-authored-by: Cursor <cursoragent@cursor.com>
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 25 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (2)
Comment |
Summary
bazeliskinpre_push_validationpreflight andmake pre-push-fastcargo-bazel-drift-check.pyon the fast pathmake bazel-testfor optional local//:ci_rust_testsTest plan
python3 scripts/ci/cargo-bazel-drift-check.pypython3 scripts/ci/test-pre-push-validation.pymake -n bazel-test/make -n pre-push-fastCloses #447
Made with Cursor
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Note
Add Bazelisk presence check and Cargo/Bazel drift check to
pre-push-fastmake pre-push-fastnow checks forbazeliskon PATH and runsscripts/ci/cargo-bazel-drift-check.pybefore format/lint/security checks, failing fast if either prerequisite is not met.make bazel-testtarget that runsbazelisk test //:ci_rust_tests, the authoritative CI Rust test suite.scripts/pre_push_validation.pypreflight check is extended to requirebazeliskon PATH, with error messages pointing to bazel.md.bazeliskinstalled will now seemake pre-push-fastfail where it previously succeeded.Macroscope summarized e2f996d.