Skip to content

fix(ci): enable allow_auto_merge and convert dependabot-automerge to thin-caller stub#223

Merged
don-petry merged 2 commits into
mainfrom
claude/issue-107-20260508-1732
May 10, 2026
Merged

fix(ci): enable allow_auto_merge and convert dependabot-automerge to thin-caller stub#223
don-petry merged 2 commits into
mainfrom
claude/issue-107-20260508-1732

Conversation

@don-petry
Copy link
Copy Markdown
Contributor

Summary

  • Enabled the allow_auto_merge repository setting via GitHub API (nulltrue), resolving the weekly compliance audit finding
  • Converted .github/workflows/dependabot-automerge.yml from an inline workflow to the standard thin-caller stub that delegates to dependabot-automerge-reusable.yml@v1

Why both changes

The compliance audit (scripts/compliance-audit.sh) checks allow_auto_merge via the GitHub API and was seeing null. The setting has now been applied directly.

The inline dependabot-automerge.yml was pre-centralization code that:

  • Was missing skip-commit-verification: true (present in the reusable since a later update)
  • Was missing the "Check app secrets" step (present in the reusable)
  • Duplicated eligibility logic already maintained in dependabot-automerge-reusable.yml

The thin-caller stub is the org standard per standards/workflows/dependabot-automerge.yml and delegates all logic to the reusable.

Test plan

  • allow_auto_merge verified true via gh api repos/petry-projects/.github --jq '.allow_auto_merge'
  • Stub calls dependabot-automerge-reusable.yml@v1 which has full eligibility logic, skip-commit-verification: true, and app secrets check
  • Next weekly compliance audit should close issue Compliance: allow_auto_merge #107 automatically (no longer finds null)
  • Next Dependabot PR to this repo will exercise the reusable path

Closes #107

Generated with Claude Code

…allow_auto_merge

- Replace inline dependabot-automerge.yml with the standard thin-caller
  stub that delegates to dependabot-automerge-reusable.yml@v1. The inline
  version was missing skip-commit-verification: true (added in the
  reusable) and duplicated eligibility logic already maintained centrally.
- The allow_auto_merge repository setting has been enabled via API to
  satisfy the compliance audit requirement (was null, now true). The
  setting is required for gh pr merge --auto calls in the automerge
  workflow to succeed.

Closes #107

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 8, 2026 17:40
Copilot AI review requested due to automatic review settings May 8, 2026 17:40
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 8, 2026

Warning

Rate limit exceeded

@github-actions[bot] has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 59 minutes and 59 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: ASSERTIVE

Plan: Pro

Run ID: 9e23848e-0b71-4ee3-bf82-eefa89e075d2

📥 Commits

Reviewing files that changed from the base of the PR and between fc73ff1 and 1e1a099.

📒 Files selected for processing (1)
  • .github/workflows/dependabot-automerge.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-107-20260508-1732

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.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

@don-petry
Copy link
Copy Markdown
Contributor Author

Self-Review

The changes look correct. One item needs human attention:

SonarCloud Security Hotspots

SonarCloud flagged 2 security hotspots related to the pull_request_target trigger combined with secrets: inherit. These are expected patterns for this type of workflow and are safe here because the reusable workflow never checks out untrusted pull request code — it only runs gh pr review and gh pr merge commands using an authenticated App token.

These hotspots appear because this file uses the verbatim org standard template from standards/workflows/dependabot-automerge.yml, which mandates both pull_request_target and secrets: inherit. Every repo that adopts this standard will have the same hotspots.

The SonarCloud Code Analysis check is not a required status check for this repo (code-quality ruleset is not yet configured for .github), so this does not block the merge.

Action needed: Please acknowledge/dismiss these hotspots as "Safe" in the SonarCloud dashboard after reviewing.


@petry-projects/org-leads — please review and merge when ready.

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@don-petry don-petry merged commit 59592bb into main May 10, 2026
3 checks passed
@don-petry don-petry deleted the claude/issue-107-20260508-1732 branch May 10, 2026 00:55
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: allow_auto_merge

2 participants