Commit ea41f945 authored by Thong Kuah's avatar Thong Kuah

Merge branch 'jeromezng-master-patch-43083' into 'master'

Add secure coding guidelines to definition of done

See merge request gitlab-org/gitlab!25672
parents 96df25d8 3546475a
...@@ -222,6 +222,7 @@ requirements. ...@@ -222,6 +222,7 @@ requirements.
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.
1. [Secure coding guidelines](https://gitlab.com/gitlab-com/gl-security/security-guidelines) have been followed.
1. [Documented](../documentation/index.md) in the `/doc` directory. 1. [Documented](../documentation/index.md) in the `/doc` directory.
1. [Changelog entry added](../changelog.md), if necessary. 1. [Changelog entry added](../changelog.md), if necessary.
1. Reviewed by relevant (UX/FE/BE/tech writing) reviewers and all concerns are addressed. 1. Reviewed by relevant (UX/FE/BE/tech writing) reviewers and all concerns are addressed.
......
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