imsky / imsky/pull-review

Enable excluding initial and subsequent reviewers for a PR when retrying review

Open
#104 0 comments 0 reactions 0 assignees View on GitHub
feature
Dominant language
JavaScript
Stars
210
Forks
15
PR merge metrics
No merged PRs in 30d

Description

it's straightforward for a PR to get stuck in a review loop as the author issues "review ... again" over and over, cycling between the same set of reviewers. it'd be useful to store an association of reviewers to PRs in memory, with a configurable TTL, so that if a reviewer has been recently assigned, they can be excluded for the next retry.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the code that handles the "review ... again" retry and assigns pull request reviewers. Trace how reviewer assignments are selected, then add the in-memory PR association and configurable TTL so recently assigned reviewers are excluded on the next retry; done means repeated retries can avoid the same reviewers.

Written by the indexing model from the issue text.

Assessment

Tech stack
github, javascript
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.