Skip to content

update pr dashboard readme#18

Merged
trask merged 1 commit into
open-telemetry:mainfrom
maryliag:pr-info
Jun 26, 2026
Merged

update pr dashboard readme#18
trask merged 1 commit into
open-telemetry:mainfrom
maryliag:pr-info

Conversation

@maryliag

Copy link
Copy Markdown
Contributor

Add description of all columns on the dashboard.

@trask can you make sure I got the Age column correct?

Copilot AI review requested due to automatic review settings June 26, 2026 12:34
@maryliag maryliag requested a review from a team as a code owner June 26, 2026 12:34

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.

Pull request overview

This PR adds a "Dashboard columns" section to the PR dashboard README, documenting how the dashboard groups/sorts pull requests and explaining each of the six table columns (PR, Author, Reviewers, CI, Conflicts, Age), including the icon legends and the Age computation logic. It is a documentation-only change that improves discoverability of behavior already implemented in the dashboard scripts.

Changes:

  • Documents the grouping ("Waiting on …" routes), draft-PR placement, and longest-waiting-first ordering.
  • Documents per-column meanings and icon legends for Reviewers, CI, and Conflicts.
  • Documents the Age column's "time the routed party has been waiting" semantics, its fallback priority order, and its display format.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@trask

trask commented Jun 26, 2026

Copy link
Copy Markdown
Member

oh, just saw this (and I just sent #19), I'll merge this first, thanks!

@trask trask merged commit 34186be into open-telemetry:main Jun 26, 2026
5 checks passed
@maryliag maryliag deleted the pr-info branch June 26, 2026 16:57
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