Skip to content

fix: detect reviewers from actual reviews, not just explicit requests#11

Merged
rjan90 merged 1 commit intomasterfrom
phi/pr-notifier-nits
Feb 13, 2026
Merged

fix: detect reviewers from actual reviews, not just explicit requests#11
rjan90 merged 1 commit intomasterfrom
phi/pr-notifier-nits

Conversation

@rjan90
Copy link
Contributor

@rjan90 rjan90 commented Feb 13, 2026

From feedback in Slack.

  • Detect reviewers via latestReviews in addition to reviewRequests, so people who commented/reviewed without being explicitly assigned show up in the "Awaiting Your Review" list
  • Exclude dependabot PRs from the "No Reviewer Assigned" section

In the example from the Slack thread:
For PR 247, BigLep's COMMENTED review will now be detected, causing the PR to, Show up in BigLep's "PRs Awaiting Your Review" list, and no longer appear in "No Reviewer Assigned" section

fix: detect reviewers from actual reviews, not just explicit requests
@rjan90 rjan90 requested a review from BigLep February 13, 2026 08:38
@rjan90 rjan90 self-assigned this Feb 13, 2026
@rjan90 rjan90 added this to the M4.1: mainnet ready milestone Feb 13, 2026
@rjan90 rjan90 added this to FOC Feb 13, 2026
@github-project-automation github-project-automation bot moved this to 📌 Triage in FOC Feb 13, 2026
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting review in FOC Feb 13, 2026
@github-project-automation github-project-automation bot moved this from 🔎 Awaiting review to ✔️ Approved by reviewer in FOC Feb 13, 2026
@rjan90 rjan90 merged commit cd4911c into master Feb 13, 2026
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FOC Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

2 participants