Skip to content

Improve pull request dashboard docs#19

Merged
trask merged 1 commit into
open-telemetry:mainfrom
trask:copilot-token-usage-dashboard
Jun 26, 2026
Merged

Improve pull request dashboard docs#19
trask merged 1 commit into
open-telemetry:mainfrom
trask:copilot-token-usage-dashboard

Conversation

@trask

@trask trask commented Jun 26, 2026

Copy link
Copy Markdown
Member

Clarify the pull request dashboard docs by describing the next-action queues, incremental webhook updates, hourly full-run backstop, classification cache behavior, and manual workflow run guidance.

@trask trask force-pushed the copilot-token-usage-dashboard branch from 20c5e91 to 8c0904b Compare June 26, 2026 16:27
@trask trask changed the title Document pull request dashboard behavior Improve pull request dashboard docs Jun 26, 2026
@trask trask marked this pull request as ready for review June 26, 2026 16:28
@trask trask requested a review from a team as a code owner June 26, 2026 16:28
Copilot AI review requested due to automatic review settings June 26, 2026 16:28
@trask trask mentioned this pull request Jun 26, 2026

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 is a documentation-only update that clarifies how the Pull Request Dashboard shared workflow operates. It adds a new "What the dashboard tracks" section describing the next-action queues, documents the webhook-incremental / hourly-full-run model and classification cache behavior, and rewrites the manual-run guidance. I verified the prose against the implementation: the four queues match render.py (maintainer/approver/author/external), drafts are rendered in a separate section, the schedule is hourly (cron: "0 * * * *"), and the workflow_dispatch repository input behaves as described. The renamed ## Manual full run heading matches its #manual-full-run anchor reference, and no stale #manual-runs references remain.

Changes:

  • Added "What the dashboard tracks" section and notes on webhook incremental runs, hourly full-run backstop, and classification-cache token savings in README.md.
  • Renamed/rewrote the "Manual runs" section into "Manual full run" with clearer step-by-step guidance, and tightened wording throughout.
  • Added RATIONALE bullets clarifying per-repository state namespacing and cached-classification reuse for unchanged review threads.

Reviewed changes

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

File Description
pull-request-dashboard/README.md Clarifies dashboard queues, run model, opt-in steps, and manual-run instructions; wording cleanup
.github/scripts/pull-request-dashboard/RATIONALE.md Adds notes on state namespacing and cached-classification reuse

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

@trask trask force-pushed the copilot-token-usage-dashboard branch 2 times, most recently from 5b4ce84 to 66f9aab Compare June 26, 2026 16:33
@trask trask force-pushed the copilot-token-usage-dashboard branch from 66f9aab to 7e34b45 Compare June 26, 2026 16:33
Comment thread pull-request-dashboard/README.md

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 2 out of 2 changed files in this pull request and generated no new comments.

@maryliag maryliag 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.

Nice!

@trask trask merged commit a83d0ec into open-telemetry:main Jun 26, 2026
5 checks passed
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