summaryrefslogtreecommitdiff
path: root/CONTRIBUTING.rst
diff options
context:
space:
mode:
authorLaurence Urhegyi <laurence.urhegyi@codethink.co.uk>2019-02-22 16:24:51 +0000
committerbst-marge-bot <marge-bot@buildstream.build>2019-02-22 16:24:51 +0000
commit48c47bfa4adf0665312f9134104b0b8b41358901 (patch)
tree7f95dec2db8c68693dda4f4ddfa70f4449475ffa /CONTRIBUTING.rst
parent9d1c0d03ae7097f2d0c75c775e7f8b3643a75230 (diff)
downloadbuildstream-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.rst9
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
~~~~~~~~~~~~~~~~~