feat: timeline-event check for old-head review + CLEAN merge decisions#66
Conversation
A follow-up push (docs-only changes often do not re-trigger Copilot) leaves the only review on a previous commit while the ruleset reports CLEAN off it. The reviews list and reviewRequests cannot say whether a new review was announced after the push - the issue timeline can. Document the review_requested/reviewed timeline query and the two outcomes (no announcement after the push = mergeable; announcement without a landed review = wait). Applied three times during the nr-repurpose v0.1.0 release day. Adds eval old_head_review_clean_timeline_check. Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds a new evaluation test case and updates the documentation to detail how to determine mergeability when a review exists on an earlier commit head and the status is CLEAN. The reviewer suggested replacing the OWNER/REPO placeholder in the GitHub CLI command with an explicit repository variable ($R) to ensure compatibility outside the local git context.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
…ge-gate style) Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
|
Companion to the arming gate and the 'never merge over an announced review' rule: after a follow-up push, the only review on record may sit on the previous commit while
mergeStateStatusreportsCLEANoff it (areview_on_push: falseruleset).reviewRequests: []and the reviews list cannot distinguish 'no new review will come' from 'one is in flight'.The new subsection documents the deciding query — the issue timeline's
review_requested/reviewedevents compared against the latest push time — and both outcomes. Used three times during the nr-repurpose v0.1.0 release day (2026-06-12), incl. a docs-only push that did not re-trigger Copilot.Eval
old_head_review_clean_timeline_checkadded.