ci: delete redundant bot-branch-delete workflow; make cleanup explicit#26
Conversation
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>
|
Warning Review limit reached
More reviews will be available in 43 minutes and 6 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 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 configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
This PR streamlines bot PR branch cleanup by deleting bot PR head branches explicitly at merge time and removing a redundant on-close cleanup workflow that could race and fail.
Changes:
- Add
--delete-branchto thegh pr merge --auto --squashcommands inauto-merge-bots.yml. - Remove the redundant
delete-merged-bot-branches.ymlworkflow that attempted to delete already-deleted branches onpull_request: closed.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.github/workflows/auto-merge-bots.yml |
Makes bot PR merges explicitly delete the head branch at merge time via --delete-branch. |
.github/workflows/delete-merged-bot-branches.yml |
Deletes the redundant on-close branch deletion workflow to avoid races and “Reference does not exist” failures. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
Applies the same fix merged in
SrrDBAutomationToolkit#29to this repo.--delete-branchto bothgh pr merge --auto --squashcalls inauto-merge-bots.yml, so bot PR head branches are deleted explicitly atmerge time (author-independent, in-code).
delete-merged-bot-branches.ymlon-close workflow.It fired on
pull_request: closedand re-deleted a branch that was alreadygone, racing merge-time cleanup and only ever failing with
Reference does not exist.Breaking Changes
None.