Skip to content

Redesign PR dashboard status comment layout#145

Open
trask wants to merge 5 commits into
open-telemetry:mainfrom
trask:pr-status-comment-layout
Open

Redesign PR dashboard status comment layout#145
trask wants to merge 5 commits into
open-telemetry:mainfrom
trask:pr-status-comment-layout

Conversation

@trask

@trask trask commented Jul 22, 2026

Copy link
Copy Markdown
Member

Reworks the dashboard-managed PR status comment from the Waiting on: / Next step: label stack into a bold status headline with inline freshness, states the ask as a sentence (with feedback links nested only when there are items), and moves the reroute/report/staleness guidance into a collapsible <details> footer so it's available without dominating every comment.

Bumps STATUS_COMMENT_REVISION to 14 so the new layout rolls out to open PRs on the next hourly run.

Example (author route, failing checks + review feedback; links are placeholders):

Pull request dashboard status

Waiting on the author · refreshed 2026-07-22 20:36 UTC

Two things need attention:

  • Required checks are failing — investigate the failures.
  • 3 review items — respond to each (e.g. link a commit, explain why not, ask a follow-up):
    • Inline threads: 1, 2
    • Top-level threads: 3
Doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Should this be with reviewers? Comment /dashboard route:reviewers to route it to them.
  • Anything wrong — including the routing? Report it with what you expected; it helps us improve the dashboard.

Replace the Waiting on/Next step label stack with a bold status headline
plus inline freshness, state the ask as a sentence, and move the
reroute/report/staleness guidance into a collapsible details footer.

Bumps STATUS_COMMENT_REVISION to 14 to roll the new layout out to open PRs.
@trask
trask requested a review from Copilot July 22, 2026 22:31
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 22, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-26 18:59:26 UTC.

  • Waiting on: Maintainers
  • Next step: Merge when ready.

This automated status or its linked feedback items may be incorrect. If something looks wrong, please report it with the result you expected.

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

Redesigns dashboard PR status comments for clearer status, actions, and secondary guidance.

Changes:

  • Adds inline status headlines and freshness timestamps.
  • Moves rerouting, reporting, and staleness guidance into a collapsible footer.
  • Updates route presentation data, tests, and rollout revision.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
pr_status_comment.py Implements the redesigned comment rendering.
route_presentation.py Adds route-specific status headlines.
test_pr_status_comment.py Updates expectations for the new layout.

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

Comment thread .github/scripts/pull-request-dashboard/pr_status_comment.py

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@trask
trask marked this pull request as ready for review July 22, 2026 22:47
@trask
trask requested a review from a team as a code owner July 22, 2026 22:47
@trask
trask requested a review from adrielp July 22, 2026 22:47

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

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.

3 participants