Skip to content

Clarify Verify:Testing team code review process

Ricky Wiens requested to merge rw/clarify-code-review into master

Why is this change being made?

There was some confusion around the process of assigning people for code review for grouptesting Merge Requests. This clarifies the intent of the process.

Author Checklist

  • Provided a concise title for the MR
  • Added a description to this MR explaining the reasons for the proposed change, per say-why-not-just-what
  • Assign this change to the correct DRI
    • If the change relates to any part of the project other than updates to content and/or data files (e.g. team.yml) please make sure to ping @gl-static-site-editor in a comment for a review and merge.
    • If the DRI for the page/s being updated isn't immediate clear, then assign it to your manager.
    • If your manager does not have merge rights, please ask someone to merge it AFTER it has been approved by your manager in #mr-buddies

For help with failing pipelines reach out in #mr-buddies in Slack

Merge request reports