1*4bcd95a3SBarry Smith(mr_management)= 2*4bcd95a3SBarry Smith 3*4bcd95a3SBarry Smith# Merge request management 4*4bcd95a3SBarry Smith 5*4bcd95a3SBarry SmithAt any given time, at least one of the {any}`sec_core_developers` is an 6*4bcd95a3SBarry Smith"MR Shepherd" and ensures that open merge 7*4bcd95a3SBarry Smithrequests progress through the {any}`review process <sec_mr_reviewing>`, 8*4bcd95a3SBarry Smithexamining open merge requests and taking appropriate action. 9*4bcd95a3SBarry Smith 10*4bcd95a3SBarry Smith```{eval-rst} 11*4bcd95a3SBarry Smith.. list-table:: MR Shepherd Checks 12*4bcd95a3SBarry Smith :widths: 50 50 13*4bcd95a3SBarry Smith :align: left 14*4bcd95a3SBarry Smith :header-rows: 1 15*4bcd95a3SBarry Smith 16*4bcd95a3SBarry Smith * - MR State 17*4bcd95a3SBarry Smith - Action 18*4bcd95a3SBarry Smith * - Missing a :any:`workflow label <sec_workflow_labels>` and other labels 19*4bcd95a3SBarry Smith - Add an appropriate label, or label ``workflow::Waiting-on-Submitter`` and ask the submitter/assignee to update 20*4bcd95a3SBarry Smith * - Without an assignee 21*4bcd95a3SBarry Smith - Assign the submitter (if the MR is from a fork, also list an appropriate developer) 22*4bcd95a3SBarry Smith * - Without reviewers 23*4bcd95a3SBarry Smith - Assign reviewers 24*4bcd95a3SBarry Smith``` 25*4bcd95a3SBarry Smith 26*4bcd95a3SBarry SmithIf MRs are inactive for too long, remind the submitter/assignee, reviewer(s), or integrator(s) of actions to take. 27*4bcd95a3SBarry SmithIf the submitter/assignee must take action, change the label to `workflow::Waiting-on-Submitter`. 28*4bcd95a3SBarry Smith 29*4bcd95a3SBarry Smith```{eval-rst} 30*4bcd95a3SBarry Smith.. list-table:: MR Inactivity Thresholds 31*4bcd95a3SBarry Smith :widths: 50 50 32*4bcd95a3SBarry Smith :align: left 33*4bcd95a3SBarry Smith :header-rows: 1 34*4bcd95a3SBarry Smith 35*4bcd95a3SBarry Smith * - MR state 36*4bcd95a3SBarry Smith - Inactivity threshold 37*4bcd95a3SBarry Smith * - ``workflow:Pipeline-Testing`` 38*4bcd95a3SBarry Smith - One week 39*4bcd95a3SBarry Smith * - ``workflow::Review`` 40*4bcd95a3SBarry Smith - One week 41*4bcd95a3SBarry Smith * - ``workflow::Ready-for-Merge`` 42*4bcd95a3SBarry Smith - Three days 43*4bcd95a3SBarry Smith * - ``workflow::Waiting-on-Submitter`` 44*4bcd95a3SBarry Smith - One month 45*4bcd95a3SBarry Smith * - ``workflow::Request-for-Comment`` 46*4bcd95a3SBarry Smith - One month 47*4bcd95a3SBarry Smith * - ``workflow::Requires-Discussion`` 48*4bcd95a3SBarry Smith - One month 49*4bcd95a3SBarry Smith * - All others 50*4bcd95a3SBarry Smith - One year 51*4bcd95a3SBarry Smith``` 52*4bcd95a3SBarry Smith 53*4bcd95a3SBarry SmithIf a submitter has been unresponsive for a year, 54*4bcd95a3SBarry Smithclose the MR, label `workflow::Inactive-closed`, 55*4bcd95a3SBarry Smithand let the submitter know that they may reopen if desired. 56