xiaoxue_admin
/
orca
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove ready-to-merge badge from checks panel (#2846)

This commit is contained in:
Jinjing 2026-05-26 11:23:30 -07:00 committed by GitHub
parent 64af9a01c1
commit c6aa8f6e83
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/renderer/src/components/right-sidebar/ChecksPanel.tsx
View File

@ -1424,14 +1424,11 @@ export default function ChecksPanel(): React.JSX.Element {
if (activeReviewClassification.needsResponse) {
badges.push('Needs response')
}
if (activeReviewClassification.readyToMerge) {
badges.push('Ready to merge')
}
// Why: viewer/author/requestedReviewer signals are not wired into the
// ChecksPanel call site yet, so `state` and `requested` would mis-classify
// every PR (collapsing to 'teammate'). Suppress those badges until the
// inputs are available; needs-response / ready-to-merge work from PR
// metadata alone and remain accurate.
// inputs are available; needs-response works from PR metadata alone and
// remains accurate.
return badges
}, [activeReviewClassification])