openlibhums / openlibhums/janeway
Ignoring unacknowledged reviews
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 238
- Forks
- 97
- Avg merge
- 9d 1h
- Merged PRs (30d)
- 8
Description
The system doesn't currently notify editors that all reviews are complete if there are review invitations that were never acknowledged (very common). It would make sense to notify the editor when all accepted reviews are complete. In principle this would fail in the case where a super-fast reviewer submits their review before other reviewers have acknowledged their invitations, but this happens very rarely, and in this scenario the editor would quickly see what happened and just ignore it.
Another option (not mutually exclusive) would be to introduce a time-out, where unacknowledged reviews are removed after some number of weeks. It would be nice if the editor-in-chief can set this time-out.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the review invitation, accepted-review completion, and editor notification workflows. Define how unacknowledged invitations affect completion, then check whether the timeout proposal is included; done means editors are notified when all accepted reviews finish and the chosen behavior is covered by tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100