Skip to content

Update review rules - #3026

Merged
dsikka merged 1 commit into
mainfrom
change_rule
Aug 12, 2026
Merged

Update review rules#3026
dsikka merged 1 commit into
mainfrom
change_rule

Conversation

@dsikka

@dsikka dsikka commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator
  • Require one review from a maintainer, unless the two-reviews label has been applied

@dsikka dsikka added the ready When a PR is ready for full CI testing before merge label Aug 12, 2026
@github-actions

Copy link
Copy Markdown

👋 Hi! Thank you for contributing to llm-compressor. Please add the ready label when the PR is ready for review.

Note: This is required to complete the testing suite, please only add the label once the PR is code complete and local testing has been performed.

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 8db4b28b-dbaf-40fa-9535-834103d09ce6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

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

@mergify

mergify Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 📃 Configuration Change Requirements

Mergify configuration change

  • check-success = Configuration changed

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request simplifies the PR review workflow by removing the automatic 'two-reviews' labeling for community PRs and introducing a new 'Require one maintainer review' rule in Mergify. Feedback suggests clarifying or updating the conditional logic (if: []) for this new rule to ensure it interacts correctly with the 'two-reviews' label as intended.

Comment thread .github/mergify.yml

@kylesayrs kylesayrs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@dsikka
dsikka merged commit 20499a9 into main Aug 12, 2026
8 of 10 checks passed
@dsikka
dsikka deleted the change_rule branch August 12, 2026 17:53
Roderick-Wu pushed a commit that referenced this pull request Aug 17, 2026
- Require one review from a maintainer, unless the two-reviews label has
been applied

Signed-off-by: Roderick-Wu <roderickwu2003@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready When a PR is ready for full CI testing before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants