diff options
author | Laurence Urhegyi <laurence.urhegyi@codethink.co.uk> | 2019-02-22 16:24:51 +0000 |
---|---|---|
committer | bst-marge-bot <marge-bot@buildstream.build> | 2019-02-22 16:24:51 +0000 |
commit | 48c47bfa4adf0665312f9134104b0b8b41358901 (patch) | |
tree | 7f95dec2db8c68693dda4f4ddfa70f4449475ffa /CONTRIBUTING.rst | |
parent | 9d1c0d03ae7097f2d0c75c775e7f8b3643a75230 (diff) | |
download | buildstream-48c47bfa4adf0665312f9134104b0b8b41358901.tar.gz |
CONTRIBUTING.rst: Clarify WIP MR closure policy
Adds section to readme detailing the WIP MR closure policy.
Diffstat (limited to 'CONTRIBUTING.rst')
-rw-r--r-- | CONTRIBUTING.rst | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index c7ab1b8ab..8a1b7996a 100644 --- a/CONTRIBUTING.rst +++ b/CONTRIBUTING.rst @@ -105,6 +105,15 @@ Consider marking a merge request as WIP again if you are taking a while to address a review point. This signals that the next action is on you, and it won't appear in a reviewer's search for non-WIP merge requests to review. +As a general rule of thumb, after a month of no activity from the submitter of +a non-WIP MR, we'll put it back into WIP with a polite note. Then after another +month with no activity we'll close the MR off entirely with another note. +In this way we are trying to ensure all of the MRs in our backlog are relevant +and up to date. We have some `boilerplate text +<https://gitlab.com/BuildStream/buildstream/blob/master/.gitlab/merge_request_templates/stale_MR_message.md>`_, +to help us when writing these notes. + + Organized commits ~~~~~~~~~~~~~~~~~ |