Skip to content

Disable "Review skipped" comments for PRs without specified labels#143728

Merged
jozala merged 1 commit intoelastic:mainfrom
jozala:coderabbit-no-review-skipped
Mar 6, 2026
Merged

Disable "Review skipped" comments for PRs without specified labels#143728
jozala merged 1 commit intoelastic:mainfrom
jozala:coderabbit-no-review-skipped

Conversation

@jozala
Copy link
Copy Markdown
Contributor

@jozala jozala commented Mar 6, 2026

Updated .coderabbit.yml to add review_status: false, preventing unnecessary "Review skipped" comments on pull requests without matching labels.

https://docs.coderabbit.ai/reference/configuration#param-review-status

review_status
Post review status messages (e.g., when a review is skipped) in the walkthrough summary comment.

Summary by CodeRabbit

  • Chores
    • Updated configuration for code review automation processes.

Updated `.coderabbit.yml` to add `review_status: false`, preventing unnecessary "Review skipped" comments on pull requests without matching labels.
@elasticsearchmachine elasticsearchmachine added v9.4.0 needs:triage Requires assignment of a team area label labels Mar 6, 2026
@jozala jozala added >non-issue :Delivery/Tooling Developer tooliing and automation auto-backport Automatically create backport pull requests when merged branch:9.2 branch:8.19 branch:9.3 and removed needs:triage Requires assignment of a team area label v9.4.0 labels Mar 6, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 3fcd2e9c-92ff-4d58-904d-02ffd902e36c

📥 Commits

Reviewing files that changed from the base of the PR and between cea9eb7 and fc28319.

📒 Files selected for processing (1)
  • .coderabbit.yml

📝 Walkthrough

Walkthrough

Configuration file updated to add a review status flag and expand automated review labels to include a new team designation.

Changes

Cohort / File(s) Summary
Configuration Update
.coderabbit.yml
Added review_status: false field under reviews section; expanded auto_review.labels to include "Team:Search - Inference" alongside existing "Team:Delivery" label.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A config tweak, so small and neat,
New labels dance to a team beat,
Review status marked with a gentle "false,"
Settings shuffle, no logic waltz,
CodeRabbit hops with delight! 🐇✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: disabling review skipped comments by setting review_status to false in the configuration file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

@jozala jozala merged commit e864533 into elastic:main Mar 6, 2026
18 of 37 checks passed
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💔 Backport failed

Status Branch Result
9.3 Commit could not be cherrypicked due to conflicts
8.19
9.2

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 143728

jozala added a commit to jozala/elasticsearch that referenced this pull request Mar 6, 2026
…lastic#143728)

Updated `.coderabbit.yml` to add `review_status: false`, preventing unnecessary "Review skipped" comments on pull requests without matching labels.
jozala added a commit to jozala/elasticsearch that referenced this pull request Mar 6, 2026
…lastic#143728)

Updated `.coderabbit.yml` to add `review_status: false`, preventing unnecessary "Review skipped" comments on pull requests without matching labels.
elasticsearchmachine pushed a commit that referenced this pull request Mar 6, 2026
…143728) (#143731)

Updated `.coderabbit.yml` to add `review_status: false`, preventing unnecessary "Review skipped" comments on pull requests without matching labels.
elasticsearchmachine pushed a commit that referenced this pull request Mar 6, 2026
…143728) (#143730)

Updated `.coderabbit.yml` to add `review_status: false`, preventing unnecessary "Review skipped" comments on pull requests without matching labels.
spinscale pushed a commit to spinscale/elasticsearch that referenced this pull request Mar 6, 2026
…lastic#143728)

Updated `.coderabbit.yml` to add `review_status: false`, preventing unnecessary "Review skipped" comments on pull requests without matching labels.
szybia added a commit to szybia/elasticsearch that referenced this pull request Mar 6, 2026
…locations

* upstream/main: (153 commits)
  ES|QL: Update docs for TOP_SNIPPETS and DECAY (elastic#143739)
  Correctly include endpoint id in log msg in AuthorizationPoller (elastic#143743)
  Bar searching or sorting on _seq_no when disabled (elastic#143600)
  Generalize `testClientCancellation` test (elastic#143586)
  JSON_EXTRACT: zero-copy byte slicing for object, array, and number extraction (elastic#143702)
  Track recycler pages in circuit breaker (elastic#143738)
  [ESQL] Enable distributed pipeline breakers for external sources via FragmentExec (elastic#143696)
  Adding 'mode' and 'codec' fields to ES monitoring template (elastic#143673)
  [ESQL] Columnar I/O and vectorized block conversion for external sources (elastic#143703)
  Fix flaky MMR diversification YAML tests (elastic#143706)
  ES|QL codegen: check builder arguments for vector support (elastic#143724)
  Add Views Security Model (elastic#141050)
  ESQL: Prevent pushdown of unmapped fields in filters and sorts (elastic#143460)
  Don't run seq_no pruning tests in release CI (elastic#143725)
  ESQL: Support intra-row field references in ROW command (elastic#140217)
  ES|QL: Remove implicit limit in FORK branches in CSV tests (elastic#143601)
  IndexRoutingTests with and without synthetic id (elastic#143566)
  Synthetic id upgrade test in serverless (elastic#142471)
  Disable "Review skipped" comments for PRs without specified labels (elastic#143728)
  Cleanup ES|QL T-Digest code duplication, add memory accounting (elastic#143662)
  ...
sidosera pushed a commit to sidosera/elasticsearch that referenced this pull request Mar 6, 2026
…lastic#143728)

Updated `.coderabbit.yml` to add `review_status: false`, preventing unnecessary "Review skipped" comments on pull requests without matching labels.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :Delivery/Tooling Developer tooliing and automation >non-issue Team:Delivery Meta label for Delivery team v8.19.13 v9.2.7 v9.3.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants