Skip to content

Automatically size pull request dashboards - #514

Merged
trask merged 2 commits into
mainfrom
trask-dashboard-large-repositories
Nov 6, 2026
Merged

trask merged 2 commits into
mainfrom
trask-dashboard-large-repositories

Conversation

@trask

@trask trask commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Automatically fit pull request dashboards within GitHub's 65,536-character issue-body limit. Show all PRs, including drafts, whenever they fit, without a fixed 100-row cap.

When needed, trim complete rows from bottom to top: drafts, unknown PRs, transient GitHub failures, PRs waiting on authors, reviewers, then maintainers. Exhaust each lower section before trimming the next, preserving the longest-waiting PRs within each section.

Add an overall truncation warning and keep section headings with exact omission counts even when every row is hidden. Routing and notifications are unchanged.

Roll out through canary before stable promotion. Retain the deprecated large_repo setting and flag forwarding so pinned stable publishers keep their size protection; the automatic publisher accepts --large-repo as a no-op. Remove that compatibility support after promotion.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@trask
trask requested a balanced review from Copilot November 3, 2026 23:30
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-11-06 16:42 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.

🟡 Changes recommended

Removing the large-repo setting and flag forwarding disables sizing protection in the pinned stable publisher, causing oversized dashboards to fail publication until the autosizing release is promoted.

1 open issue
What changed in this PR

This PR replaces manual dashboard sizing presets with automatic trimming to fit GitHub’s 65,536-character issue-body limit, while leaving routing and notifications unchanged.

Changes:

  • Show all rows, including drafts, when they fit; otherwise trim complete rows from lower-priority sections first.
  • Preserve section headings and exact omission counts, and add a truncation warning.
  • Remove large_repo configuration and CLI handling, with updated documentation and tests.
File Description
pull-request-dashboard/​README.md Documents automatic sizing and omission behavior.
.github/​scripts/​pull-request-dashboard/​test_render.py Tests size boundaries, trimming order, and omission notices.
.github/​scripts/​pull-request-dashboard/​test_publish_dashboard.py Updates publisher tests and covers draft-fetch skipping and size guards.
.github/​scripts/​pull-request-dashboard/​test_owned_execution.py Updates configuration fixtures and publisher command assertions.
.github/​scripts/​pull-request-dashboard/​repositories.json Removes collector-contrib’s manual sizing setting.
.github/​scripts/​pull-request-dashboard/​render.py Implements character-budget trimming and section omission tracking.
.github/​scripts/​pull-request-dashboard/​publish_dashboard.py Uses automatic sizing and skips unnecessary draft metadata fetches.
.github/​scripts/​pull-request-dashboard/​process_queue_batch.py Stops forwarding the legacy sizing flag.

Review effort: Balanced

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

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

No active review issues block the automatic dashboard sizing change.

1 open issue

Review effort: Balanced

@trask
trask requested a balanced review from Copilot November 4, 2026 00:20
@trask
trask marked this pull request as ready for review November 4, 2026 00:20
@trask
trask requested a review from a team as a code owner November 4, 2026 00:20
@trask
trask requested a review from mx-psi November 4, 2026 00:21

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

No active review issues are recorded for the automatic dashboard-sizing change.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@trask
trask merged commit 7bfe165 into main Nov 6, 2026
8 checks passed
@trask
trask deleted the trask-dashboard-large-repositories branch November 6, 2026 16:42
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.

5 participants