diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-03-01 06:12:27 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-03-01 06:12:27 +0300 |
commit | b0ad241fbbdf6a9d4ccb6f44f2c233d542b3b08d (patch) | |
tree | 05d1c9682c9dfdbe2ecd480e441c192dc07e50f1 /doc/development/code_review.md | |
parent | d40f85a4030015db726d2e1bea1f2ab1e16d1eb1 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'doc/development/code_review.md')
-rw-r--r-- | doc/development/code_review.md | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/doc/development/code_review.md b/doc/development/code_review.md index 245fb2152cd..c7811e16551 100644 --- a/doc/development/code_review.md +++ b/doc/development/code_review.md @@ -718,12 +718,7 @@ Enterprise Edition instance. This has some implications: cached value returns (say, from a string or nil to an array), change the cache key at the same time. 1. **Settings** should be added as a - [last resort](https://about.gitlab.com/handbook/product/product-principles/#convention-over-configuration). - If you're adding a new setting in `gitlab.yml`: - 1. Try to avoid that, and add to `ApplicationSetting` instead. - 1. Ensure that it is also - [added to Omnibus](https://docs.gitlab.com/omnibus/settings/gitlab.yml#adding-a-new-setting-to-gitlabyml). - 1. Ensure that it is also [added to Charts](https://docs.gitlab.com/charts/development/style_guide.html), if needed. + [last resort](https://about.gitlab.com/handbook/product/product-principles/#convention-over-configuration). See [Adding a new setting to GitLab Rails](architecture.md#adding-a-new-setting-in-gitlab-rails). 1. **File system access** is not possible in a [cloud-native architecture](architecture.md#adapting-existing-and-introducing-new-components). Ensure that we support object storage for any file storage we need to perform. For more information, see the [uploads documentation](uploads/index.md). |