Skip to content

Nudge authors when pull requests are waiting on them#125

Closed
trask wants to merge 3 commits into
open-telemetry:mainfrom
trask:issue-26-author-nudge
Closed

Nudge authors when pull requests are waiting on them#125
trask wants to merge 3 commits into
open-telemetry:mainfrom
trask:issue-26-author-nudge

Conversation

@trask

@trask trask commented Jul 20, 2026

Copy link
Copy Markdown
Member

Resolves #26

Add a single automatic reminder when a pull request remains in Waiting on authors for one week. The reminder mentions the author and links to the dashboard-managed status comment. Leaving the route before the reminder resets the clock, and each PR is reminded at most once. Hourly dashboard runs deliver due reminders when the pull request is next refreshed.

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 20, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-21 01:21:08 UTC.

  • Status: Closed.

This automated status or its linked feedback items may be incorrect. If something looks wrong, 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

Adds one-time author reminders for pull requests remaining in Waiting on authors for one week.

Changes:

  • Tracks waiting periods and lifetime nudge state.
  • Posts reminders during hourly refreshes with author mentions and status links.
  • Adds workflow integration, documentation, and tests.

Reviewed changes

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

Show a summary per file
File Description
pull-request-dashboard/README.md Documents reminder behavior.
.github/workflows/pull-request-dashboard-repo.yml Adds nudge processing job.
.github/scripts/pull-request-dashboard/state.py Persists nudge state.
.github/scripts/pull-request-dashboard/dashboard.py Outputs refreshed PR numbers.
.github/scripts/pull-request-dashboard/author_nudge.py Implements tracking and delivery.
.github/scripts/pull-request-dashboard/test_state.py Tests nudge-state persistence.
.github/scripts/pull-request-dashboard/test_dashboard.py Tests refreshed-PR output.
.github/scripts/pull-request-dashboard/test_author_nudge.py Tests nudge policy and posting.

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

Comment thread .github/workflows/pull-request-dashboard-repo.yml Outdated
Comment thread .github/scripts/pull-request-dashboard/author_nudge.py Outdated

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

Comment thread .github/scripts/pull-request-dashboard/dashboard.py
Comment thread .github/scripts/pull-request-dashboard/author_nudge.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 8 out of 8 changed files in this pull request and generated no new comments.

@trask

trask commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

Superseded by #129, which combines this change with #126 and moves all target-repository writes into the durable publish-dashboard job.

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.

PR Dashboard: Nudge authors when PRs are waiting on them

2 participants