Repository navigation
Conversation
## Summary - Adds the two merge-bot `Proof:` items #2412 asks for under "Stage 1: Merge-Bot Adoption" in `docs/reusable-workflows.md`, one for the `rules` input end to end and one for the `merge-app` job. - Both name homeassistant-purpleair, whose `check-ha-version.yml` tracker opens App pull requests from `ha-version-bump/matrix` that only its stub's `rules` entry matches. Both are ticked with the run that produced the proof, the `Merge App pull request job` run for ptr727/homeassistant-purpleair#417, the first tracker pull request after the stub landed in ptr727/homeassistant-purpleair#371. - Updates the "What a Pilot Proves" sentence and the matching `TODO.md` detail so neither points at #2412 any more, and drops the now unused `issue-2412` link 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](https://claude.com/claude-code) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Updated rollout notes with completed end-to-end proofs from PhotoCleaner and homeassistant-purpleair. * Clarified which adopter verifies each of four behaviors and linked to the homeassistant-purpleair proof run. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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
Only a minor wording nit remains, with no blocking issues.
0 open findings
What changed in this PR
Promotes merge-bot proof documentation and removes obsolete issue references.
Changes:
- Adds verified
rulesandmerge-appproof evidence. - Updates
TODO.mdto track all four proofs. - Removes references to issue #2412.
Review note: one minor wording nit was identified in docs/reusable-workflows.md.
| File | Summary |
|---|---|
TODO.md |
Updates merge-bot proof tracking. |
docs/reusable-workflows.md |
Records completed proof evidence and workflow-run reference. |
🧠 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 @@
## main #2666 +/- ##
=======================================
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 round names no line, sentence, or wording for the nit, so there is nothing in it to act on or disprove. The same content was reviewed on its feature pull request, #2665, where Copilot's one named nit on this file (use reference links for cited pull requests) is answered with evidence in this comment. This promotion carries that one change and nothing else, so its diff is the reviewed one. |
Promotes the one change develop holds over main:
docs/reusable-workflows.mdgains the merge-botProof:items for therulesinput and themerge-appjob, and neither it norTODO.mdpoints at #2412 any more (#2665).Fixes #2412
🤖 Generated with Claude Code