-
Notifications
You must be signed in to change notification settings - Fork 256
Description
Is your feature request related to a problem? Please describe.
I've observed in https://github.com/Project-MONAI/MONAILabel/pulls a varying method of indicating that a pull request still has some work to do or that it should not be merged. There is usage of "[WIP]" in the PR title which some GitHub apps can use by a status check, though there is no status check for this repo. There is a label called "PR_DO_NOT_MERGE" and also a label "WORK_IN_PROGRESS". In the Contributing.md there isn't a clear indication of a preferred method.

Describe the solution you'd like
Instead of the various methods, I would suggest the more robust solution of using GitHub's tool of having "Draft" PRs (https://github.blog/changelog/2020-04-08-convert-pull-request-to-draft/). This functionality can be used to indicate that there is still "Work in Progress" and that no review is being requested at this time (see image below). Also it can be used when there is an approving review to prevent merging. Labels or PR text by their own cannot prevent actual merging and rely on humans reading and following instructions. It appears that the MONAI toolkit has already adopted this method https://github.com/Project-MONAI/MONAI/pulls.
Describe alternatives you've considered
Another method is to use https://github.com/apps/wip, which when it detects "WIP" in the PR title, will set a status check to "in progress" which will prevent merging (set this as a required check) even when there is an approving review. Admin's can of course override this check.

