Skip to content

ci: delete redundant bot-branch-delete workflow; make cleanup explicit#42

Merged
tablackburn merged 1 commit into
mainfrom
ci/explicit-bot-branch-cleanup
Jun 12, 2026
Merged

ci: delete redundant bot-branch-delete workflow; make cleanup explicit#42
tablackburn merged 1 commit into
mainfrom
ci/explicit-bot-branch-cleanup

Conversation

@tablackburn

Copy link
Copy Markdown
Owner

Summary

Applies the same fix merged in SrrDBAutomationToolkit#29 to this repo.

  • Add --delete-branch to both gh pr merge --auto --squash calls in
    auto-merge-bots.yml, so bot PR head branches are deleted explicitly at
    merge time (author-independent, in-code).
  • Remove the redundant delete-merged-bot-branches.yml on-close workflow.
    It fired on pull_request: closed and re-deleted a branch that was already
    gone, racing merge-time cleanup and only ever failing with
    Reference does not exist.

Breaking Changes

None.

Mirror the change from SrrDBAutomationToolkit#29:

- Add --delete-branch to both gh pr merge calls in auto-merge-bots.yml so
  bot PR branches are deleted explicitly at merge time.
- Remove the redundant delete-merged-bot-branches.yml on-close workflow,
  which raced with merge-time cleanup and only ever failed with
  "Reference does not exist".

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 12, 2026 14:12
@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@tablackburn, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 43 minutes and 27 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more credits in the billing tab to continue.

⌛ How to resolve this issue?

After more reviews become available, 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1e56608f-16fe-4b12-9609-94c73e632510

📥 Commits

Reviewing files that changed from the base of the PR and between ff1429c and f380243.

📒 Files selected for processing (2)
  • .github/workflows/auto-merge-bots.yml
  • .github/workflows/delete-merged-bot-branches.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/explicit-bot-branch-cleanup

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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR simplifies bot-PR branch cleanup by deleting bot head branches explicitly at merge time and removing a redundant post-close cleanup workflow that could race and fail after the branch was already removed.

Changes:

  • Add --delete-branch to the gh pr merge --auto --squash commands for Dependabot and pre-commit.ci PRs.
  • Remove the pull_request: closed workflow that attempted to delete already-merged bot branches.

Reviewed changes

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

File Description
.github/workflows/delete-merged-bot-branches.yml Removes redundant post-close branch deletion job for bot PRs.
.github/workflows/auto-merge-bots.yml Deletes bot PR head branches as part of the merge configuration via gh pr merge --delete-branch.

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

@tablackburn tablackburn merged commit 0caeb3c into main Jun 12, 2026
12 checks passed
@tablackburn tablackburn deleted the ci/explicit-bot-branch-cleanup branch June 12, 2026 14:20
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