-
Suggestion
-
Resolution: Duplicate
-
None
-
None
-
None
I would like to see a small workflow on the pull request. Right now the scenario of the review works well if the pull request is ready to merge, or if the reviewer themselves are making the fix. However when the pull request has issues that need to be addressed, there should be a state to reflect this.
Something like:
1.) Log pull request
2.) Reviewer(s) nominated
3.) Review does not pass -> Rework required (this is my new suggestion). This removes the pull request from the reviewer's queue and puts it back onto the submitter's queue.
4.) Submitter fixes review comments and then marks defect as ready for review (which is new also, which would put the pull request back onto the reviewer's queue).
5.) pull request reviewed again and accepted
6.) request is then merged automatically or manually.
Right now, if I fail a review from one of my developers, then I cannot notify them via Stash that they need to review the item. Secondly the item remains in my queue, even though I have actually reviewed the issue.
- duplicates
-
BSERV-2999 Mark a pull request review as "complete" (separately from "approved")
- Closed