Repository navigation
Add Proof Items for the Merge-Bot rules Input and merge-app Job - #2665
Conversation
The rules input and the merge-app job were owed proofs with no tracker item. Both proofs now sit under Stage 1, ticked with the homeassistant-purpleair run that showed them, and the pointers that named the issue as their tracker now name those items instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The merge step enables auto-merge rather than merging inside the run, so the proof items now say that and name the merge separately. The pointer prose also says which repository each proof item names rather than which one owes it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe adoption notes now record downstream proof for four reusable-workflow behaviors. They cite PhotoCleaner for cross-repository resolution and Dependabot pin bumps, and homeassistant-purpleair for the ChangesAdoption proof documentation
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to The documentation distinguishes the run enabling auto-merge from the later PR merge. No merge-blocking issue was established. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)✨ 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.
🟢 Approval recommended
The only noted issue is a non-blocking documentation nit.
0 open findings
What changed in this PR
Adds live proof records for the merge-bot rules input and merge-app job, with related documentation updates.
Changes:
- Adds two checked Stage 1 proof items.
- Updates
TODO.mdand pilot documentation. - Removes obsolete issue references.
| File | Summary |
|---|---|
TODO.md |
Updates merge-bot proof tracking. |
docs/reusable-workflows.md |
Adds proof items and run reference. Nit (1 vote): use clickable reference links for cited pull requests. |
🧠 Review effort: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #2665 +/- ##
========================================
Coverage 59.34% 59.34%
========================================
Files 16 16
Lines 8282 8282
========================================
Hits 4915 4915
Misses 3367 3367
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Answering the one body-only note in Copilot's review round on
No change needed. The new items cite pull requests the way the rest of the Rollout tracker already does. |
|
@coderabbitai review |
✅ Action performedReview finished.
|
Promotes the one change develop holds over main: `docs/reusable-workflows.md` gains the merge-bot `Proof:` items for the `rules` input and the `merge-app` job, and neither it nor `TODO.md` points at #2412 any more (#2665). Fixes #2412 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Summary
Proof:items Add Proof Items for the Merge-Bot rules Input and merge-app Job #2412 asks for under "Stage 1: Merge-Bot Adoption" indocs/reusable-workflows.md, one for therulesinput end to end and one for themerge-appjob.check-ha-version.ymltracker opens App pull requests fromha-version-bump/matrixthat only its stub'srulesentry matches. Both are ticked with the run that produced the proof, theMerge App pull request jobrun for Bump HA test matrix (beta: 2026.10.0b2) homeassistant-purpleair#417, the first tracker pull request after the stub landed in Call the Hub's Merge-Bot Task From the Merge-Bot Workflow homeassistant-purpleair#371.TODO.mddetail so neither points at Add Proof Items for the Merge-Bot rules Input and merge-app Job #2412 any more, and drops the now unusedissue-2412link definitions.Filed along the way as #2664: the Stage 1 items the live evidence already closes, which this change leaves alone.
Closes on promotion: #2412
🤖 Generated with Claude Code
Summary by CodeRabbit