Commit 3e2beaf6 authored by Kati Paizee's avatar Kati Paizee

Merge branch 'whaber-master-optional-peer-member-testing-54766' into 'master'

Added optional peer member testing

See merge request gitlab-org/gitlab!72513
parents 31d82026 bb069daa
...@@ -240,6 +240,7 @@ requirements. ...@@ -240,6 +240,7 @@ requirements.
1. Working and clean code that is commented where needed. 1. Working and clean code that is commented where needed.
1. [Unit, integration, and system tests](../testing_guide/index.md) that all pass 1. [Unit, integration, and system tests](../testing_guide/index.md) that all pass
on the CI server. 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 1. Regressions and bugs are covered with tests that reduce the risk of the issue happening
again. again.
1. [Performance guidelines](../merge_request_performance_guidelines.md) have been followed. 1. [Performance guidelines](../merge_request_performance_guidelines.md) have been followed.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment