Skip to content

Handling of work in progress pull requests #733

@jamesobutler

Description

@jamesobutler

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.
image

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.

image

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.
image

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions