Skip to content

Hide inactive Copilot reviewers from the PR dashboard - #479

Merged
trask merged 2 commits into
mainfrom
trask-pr-dashboard-reviewers
Oct 16, 2026
Merged

trask merged 2 commits into
mainfrom
trask-pr-dashboard-reviewers

Conversation

@trask

@trask trask commented Oct 16, 2026

Copy link
Copy Markdown
Member

Show Copilot in the reviewers column only when it has unresolved inline feedback (💬) or a pending review that holds up the PR (⏳). Completed Copilot participation alone adds noise because it reviews nearly every PR.

This changes presentation only. Human reviewers, review gating, and PR routing are unchanged.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 4e5e0e0c-672d-47ce-a574-c14cd922e235
@trask
trask requested a balanced review from Copilot October 16, 2026 14:55
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-10-16 23:53 UTC

Status above doesn't look right?
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The focused display filter preserves underlying review state and has regression coverage for the affected behavior.

0 open findings

What changed in this PR

Reduces reviewer-column noise by hiding inactive Copilot entries without changing human reviewers, review gating, or PR routing.

Changes:

  • Shows Copilot only for unresolved inline feedback or a pending review that holds the PR.
  • Adds regression coverage for visibility rules and recognized Copilot logins.
  • Documents the display policy.
File Description
pull-request-dashboard/​README.md Explains when Copilot appears.
.github/​scripts/​pull-request-dashboard/​test_render.py Tests Copilot visibility and pending-review behavior.
.github/​scripts/​pull-request-dashboard/​render.py Filters inactive Copilot reviewer entries.
.github/​scripts/​pull-request-dashboard/​RATIONALE.md Records the rationale and presentation-only scope.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@trask
trask marked this pull request as ready for review October 16, 2026 14:59
@trask
trask requested a review from a team as a code owner October 16, 2026 14:59
@trask
trask requested a review from mx-psi October 16, 2026 14:59
Preserve the Copilot display rationale and the approval-based manual handoff behavior.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: c9399e63-6f28-4efb-a756-5982e91179d6
@trask
trask merged commit f09a36e into main Oct 16, 2026
6 checks passed
@trask
trask deleted the trask-pr-dashboard-reviewers branch October 16, 2026 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants