Skip to content

Assign coach in the ping-inactive-reviewers policy

Rémy Coutable requested to merge assign-coach-in-ping-inactive-reviewers into master

What does this MR do and why?

In !1453 (merged), I forgot to actually assign coach for review MRs where they would be pinged. Without that, I believe it would lead to duplicated pings every day (I'll assign the MRs where a coach was pinged in the latest run of the policy for now).

Expected impact & dry-runs

These are strongly recommended to assist reviewers and reduce the time to merge your change.

See https://gitlab.com/gitlab-org/quality/triage-ops/-/tree/master/doc/scheduled#testing-with-a-dry-run on how to perform dry-runs.

Action items

  • (If applicable) Add documentation to the handbook pages for Triage Operations =>
  • (If applicable) Identify the affected groups and how to communicate to them:
    • /cc @person_or_group =>
    • Relevant Slack channels =>
    • Engineering week-in-review
Edited by Rémy Coutable

Merge request reports