From bb069daa4bd53bb81898deb23d63b19fdeb7ef4a Mon Sep 17 00:00:00 2001 From: Wayne Haber <whaber@gitlab.com> Date: Tue, 19 Oct 2021 07:03:02 +0000 Subject: [PATCH] Added optional peer member testing --- doc/development/contributing/merge_request_workflow.md | 1 + 1 file changed, 1 insertion(+) diff --git a/doc/development/contributing/merge_request_workflow.md b/doc/development/contributing/merge_request_workflow.md index a521d89db2bf4..91643cd3ee035 100644 --- a/doc/development/contributing/merge_request_workflow.md +++ b/doc/development/contributing/merge_request_workflow.md @@ -240,6 +240,7 @@ requirements. 1. Working and clean code that is commented where needed. 1. [Unit, integration, and system tests](../testing_guide/index.md) that all pass on the CI server. +1. Peer member testing is optional but recommended when the risk of a change is high. This includes when the changes are [far-reaching](https://about.gitlab.com/handbook/engineering/development/#reducing-the-impact-of-far-reaching-work) or are for [components critical for security](../code_review.md#security). 1. Regressions and bugs are covered with tests that reduce the risk of the issue happening again. 1. [Performance guidelines](../merge_request_performance_guidelines.md) have been followed. -- GitLab