Skip to content

fix: disable check-suite auto-trigger for Claude and CodeRabbit on .github#275

Merged
donpetry-bot merged 6 commits into
mainfrom
claude/issue-274-20260513-1249
May 14, 2026
Merged

fix: disable check-suite auto-trigger for Claude and CodeRabbit on .github#275
donpetry-bot merged 6 commits into
mainfrom
claude/issue-274-20260513-1249

Conversation

@don-petry
Copy link
Copy Markdown
Contributor

@don-petry don-petry commented May 13, 2026

Summary

  • Disabled auto_trigger_checks for Claude (app_id 1236702) and CodeRabbit (app_id 347564) on petry-projects/.github via the GitHub API
  • Updated standards/github-settings.md compliance status to reflect the 2026-05-13 remediation

Root Cause

GitHub was automatically creating a queued check suite on every push for these apps. Since the apps only complete their suites when they have real work to do, orphaned queued suites were blocking auto-merge indefinitely.

What was done

The API fix was applied directly:

PATCH /repos/petry-projects/.github/check-suites/preferences
{"auto_trigger_checks": [{"app_id": 1236702, "setting": false}, {"app_id": 347564, "setting": false}]}

Result confirms both are now false:

  • app_id 1236702 (Claude): setting: false
  • app_id 347564 (CodeRabbit): setting: false

Closes #274

Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated GitHub repository settings documentation with revised compliance status and repository automation configuration notes.

Review Change Stack

…gger fix

The Claude app (app_id 1236702) and CodeRabbit (app_id 347564) auto-trigger
check suite preferences were disabled via API for petry-projects/.github,
resolving the permanently-queued check suites that were blocking auto-merge.

Closes #274

Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
@don-petry don-petry requested a review from a team as a code owner May 13, 2026 12:51
Copilot AI review requested due to automatic review settings May 13, 2026 12:51
@don-petry
Copy link
Copy Markdown
Contributor Author

@petry-projects/org-leads — this PR resolves compliance issue #274 by disabling the Claude and CodeRabbit auto-trigger check suites on this repo (applied via API). The only code change is a documentation update to standards/github-settings.md reflecting the remediation. Ready for review and merge.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3d4b0bba-7f14-4cf6-8a18-3b97386f9448

📥 Commits

Reviewing files that changed from the base of the PR and between 865cf10 and baed22c.

📒 Files selected for processing (1)
  • standards/github-settings.md

📝 Walkthrough

Walkthrough

This PR updates the "Current Compliance Status" section in standards/github-settings.md to document that check-suite auto-trigger preferences were re-applied via API for the .github repository, and updates the compliance date from 2026-05-08 to 2026-05-13 in response to issue #274.

Changes

Check-suite auto-trigger compliance status update

Layer / File(s) Summary
Check-suite auto-trigger compliance status update
standards/github-settings.md
Compliance status note updated to reflect that .github check-suite auto-trigger preferences were re-applied via API (issue #274), with compliance date bumped to 2026-05-13.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minutes

Possibly related issues

Possibly related PRs

  • petry-projects/.github#195: Updates documentation around disabling Claude/CodeRabbit check-suite auto-trigger behavior, directly aligned with this PR's compliance status recording.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 check-suite auto-trigger for Claude and CodeRabbit on the .github repository, which directly addresses the compliance issue and remediation performed.
Linked Issues check ✅ Passed The PR successfully fulfills issue #274 requirements by disabling auto-trigger for Claude (app_id 1236702) via API and documenting the remediation in standards/github-settings.md.
Out of Scope Changes check ✅ Passed The only code change is a documentation update to standards/github-settings.md reflecting the compliance remediation, which is directly in scope with issue #274.
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
  • Commit unit tests in branch claude/issue-274-20260513-1249

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
Contributor

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.

Pull request overview

Updates the GitHub settings standards documentation to reflect the remediation that disabled check-suite auto-trigger for Claude and CodeRabbit on petry-projects/.github, addressing the queued check-suite auto-merge blockage described in #274.

Changes:

  • Updated the “Current Compliance Status” timestamp to 2026-05-13.
  • Clarified the compliance note to reference the .github check-suite auto-trigger preferences being re-applied via API and tied it to issue #274.

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

Copy link
Copy Markdown
Contributor

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

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 updates the compliance status date in standards/github-settings.md to 2026-05-13 and includes a reference to issue #274. I have no further feedback to provide.

This was referenced May 13, 2026
Copy link
Copy Markdown
Contributor

@donpetry-bot donpetry-bot left a comment

Choose a reason for hiding this comment

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

Automated review — APPROVED ✓

Risk: LOW
Reviewed commit: 1d2c6f3cd4948660718d05e87c52bb4779b29395
Review mode: triage-approved (single reviewer)

Summary

Docs-only update to standards/github-settings.md: bumps the compliance status date from 2026-05-08 to 2026-05-13 and adds a reference to issue #274, documenting the API-applied remediation that disabled the Claude (app_id 1236702) and CodeRabbit (app_id 347564) auto-trigger check suites on petry-projects/.github. Two lines changed in one file; no code or configuration changes.

Linked issue analysis

Issue #274 reports that Claude's auto-trigger preference was creating permanently-queued check suites that blocked auto-merge. The PR description shows the fix was applied via PATCH /repos/petry-projects/.github/check-suites/preferences with both 1236702 and 347564 set to false, and this PR updates the standards doc to reflect that remediation. The documentation change matches the remediation described and is the appropriate paper-trail follow-up.

Findings

None. The change is a two-line factual update to a markdown compliance log; there are no security, correctness, or maintainability concerns.

CI status

All checks passing:

  • ✅ CodeQL / Analyze (actions) — SUCCESS
  • ✅ CodeQL — SUCCESS
  • ✅ CodeRabbit — SUCCESS
  • SonarCloud Quality Gate passed (0 new issues)

Merge state is BLOCKED only because REVIEW_REQUIRED — this approval should unblock it.


Reviewed automatically by the PR-review agent (single-reviewer mode: opus 4.7). Reply if you need a human review.

@donpetry-bot donpetry-bot enabled auto-merge (squash) May 14, 2026 04:46
@sonarqubecloud
Copy link
Copy Markdown

@donpetry-bot donpetry-bot merged commit 761dd4a into main May 14, 2026
22 of 23 checks passed
@donpetry-bot donpetry-bot deleted the claude/issue-274-20260513-1249 branch May 14, 2026 15:14
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.

Compliance: check-suite-auto-trigger-1236702

3 participants