Skip to content

Show human authors for Copilot draft pull requests - #484

Merged
trask merged 1 commit into
mainfrom
trask-draft-author-attribution
Oct 18, 2026
Merged

trask merged 1 commit into
mainfrom
trask-draft-author-attribution

Conversation

@trask

@trask trask commented Oct 18, 2026

Copy link
Copy Markdown
Member

Use the same author resolution for draft and non-draft dashboard rows. Copilot-authored drafts show the human assignee, falling back to the first commit's human committer when available.

Keep Copilot as the author when no human can be identified, and avoid extra commit requests when a human assignee is already available.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: fb463d90-5190-4a95-b3d4-aa7e117f2a38
@trask
trask requested a balanced review from Copilot October 18, 2026 03:23
@opentelemetry-pr-dashboard

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

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-10-18 23:40 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 change preserves existing resolution behavior, includes regression coverage, and has no unresolved findings.

No open issues 🎉

What changed in this PR

Applies the existing Copilot author-resolution policy to draft dashboard rows, keeping draft and non-draft attribution consistent.

Changes:

  • Shares author resolution between evaluation and draft publication.
  • Prefers human assignees, fetching the first commit only when needed.
  • Adds regression coverage and documents draft attribution.
File Description
pull-request-dashboard/​README.md Clarifies draft author attribution.
.github/​scripts/​pull-request-dashboard/​test_publish_dashboard.py Tests draft attribution, fallback behavior, and request avoidance.
.github/​scripts/​pull-request-dashboard/​test_github_cli.py Tests assignee preservation in PR listings.
.github/​scripts/​pull-request-dashboard/​render.py Displays effective draft authors.
.github/​scripts/​pull-request-dashboard/​pull_request_evaluation.py Uses the shared author resolver.
.github/​scripts/​pull-request-dashboard/​pull_request_author.py Extracts the existing author-resolution policy.
.github/​scripts/​pull-request-dashboard/​publish_dashboard.py Resolves draft authors before rendering.
.github/​scripts/​pull-request-dashboard/​github_cli.py Includes assignees in open-PR listings.
.github/​scripts/​pull-request-dashboard/​CONTEXT.md Documents shared resolution and draft publication behavior.

Review effort: Balanced

@trask
trask marked this pull request as ready for review October 18, 2026 03:25
@trask
trask requested a review from a team as a code owner October 18, 2026 03:25
@trask
trask requested a review from adrielp October 18, 2026 03:25
@trask
trask merged commit 2144a75 into main Oct 18, 2026
7 checks passed
@trask
trask deleted the trask-draft-author-attribution branch October 18, 2026 23:39
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