Skip to content

Start running Merge Gatekeeper when each workflow finishes#962

Merged
5ouma merged 1 commit into
mainfrom
ci-merge-gatekeeper-check-suite
May 16, 2026
Merged

Start running Merge Gatekeeper when each workflow finishes#962
5ouma merged 1 commit into
mainfrom
ci-merge-gatekeeper-check-suite

Conversation

@5ouma
Copy link
Copy Markdown
Owner

@5ouma 5ouma commented May 16, 2026

Reduce the running time of Gatekeeper.

Copilot AI review requested due to automatic review settings May 16, 2026 13:53
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 16, 2026

Review Change Stack

Warning

Rate limit exceeded

@5ouma has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 42 minutes and 36 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6d75b798-b56e-4d74-b853-b6b0a8b288f6

📥 Commits

Reviewing files that changed from the base of the PR and between 1567397 and 97cab8e.

📒 Files selected for processing (1)
  • .github/workflows/merge-gatekeeper.yml

Walkthrough

A new GitHub Actions workflow file is added to automate merge gatekeeper checks. The workflow triggers on check_suite completion for pull requests, uses minimal read permissions, and executes the merge-gatekeeper action with configuration for self-labeling and CodeRabbit exclusion.

Changes

Merge Gatekeeper Workflow

Layer / File(s) Summary
Merge Gatekeeper GitHub Actions workflow
.github/workflows/merge-gatekeeper.yml
New workflow configured to run upsidr/merge-gatekeeper on check suite completion events. Trigger is gated to pull request check suites, permissions are minimal (read-only for checks and statuses), timeout is set to 10 minutes, and the action is pinned to commit 09af7a82c1666d0e64d2bd8c01797a0bcfd3bb5d with inputs for token, self-labeling, and CodeRabbit exclusion.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a Merge Gatekeeper workflow that triggers on check_suite completion events.
Description check ✅ Passed The description mentions reducing Gatekeeper running time, which aligns with the changeset's intent to trigger Merge Gatekeeper on workflow completion.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci-merge-gatekeeper-check-suite

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 and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Warning

CodeRabbit couldn't request changes on this pull request because it doesn't have sufficient GitHub permissions.

Please grant CodeRabbit Pull requests: Read and write permission and re-run the review.

👉 Steps to fix this

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/merge-gatekeeper.yml:
- Line 16: The job condition currently uses
github.event.check_suite.pull_requests which can be an empty array (truthy);
update the condition to explicitly require a non-empty PR list by checking that
github.event.check_suite.pull_requests exists and that its length is greater
than zero (i.e., replace the existing if expression with one that asserts the
array is present and has length > 0).
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 34526eee-0efc-4751-9786-1f5fe0f6604d

📥 Commits

Reviewing files that changed from the base of the PR and between 7a99c35 and 1567397.

📒 Files selected for processing (1)
  • .github/workflows/merge-gatekeeper.yml

Comment thread .github/workflows/merge-gatekeeper.yml Outdated
@5ouma 5ouma force-pushed the ci-merge-gatekeeper-check-suite branch from 1567397 to 9dc8338 Compare May 16, 2026 14:10
@5ouma 5ouma force-pushed the ci-merge-gatekeeper-check-suite branch from 9dc8338 to 97cab8e Compare May 16, 2026 14:10
@5ouma 5ouma merged commit 9bba582 into main May 16, 2026
4 of 5 checks passed
@5ouma 5ouma deleted the ci-merge-gatekeeper-check-suite branch May 16, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants