- changed title to Add the ability to know when a PR is being actively updated
Add the ability to know when a PR is being actively updated
Issue #463
closed
USE CASE: WHAT DO YOU WANT TO DO?
I want to only review a pull request when it is not being actively changed.
STEPS TO REPRODUCE AN ISSUE (OR TRIGGER A NEW FEATURE)
- Review a PR
- Comment when review is done
- Author, seeing that the first review is done and no other reviewers are actively reviewing, decides to start working on it, so they change the name of the PR to append 'WIP', meaning "Work in Progress".
- Reviewers can now see that the state of the PR is in flux by 'WIP' being in the PR name, thus refrain from further review.
- When the author completes their current efforts, they edit the PR name to remove 'WIP'
- Reviewers see that WIP is gone and (re)commence review.
CURRENT BEHAVIOR
Reviews and work on PRs sometimes occurs concurrently, causing wasted efforts.
EXPECTED BEHAVIOR
Phases of review and updates to PRs as separated.
DEVELOPERS ONLY SECTION
SUGGESTED CHANGE (Pseudocode optional)
Add procedures to softeng docs to support the adding and removal of 'WIP' to the PR title and what developers/reviewers should do in each case.
FILES AFFECTED (where the changes will be implemented) - developers only
scmp & other documentation
LEVEL OF EFFORT - developers only
minor
COMMENTS
Comments (4)
-
reporter -
reporter - changed status to resolved
Changes are implemented in branch issue463-464-469.
-
reporter - changed status to closed
Merged to master
-
reporter - changed version to beta2
- Log in to comment