diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2019-06-25 18:34:35 +0300 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2019-06-25 18:34:35 +0300 |
commit | 75fd8a40b99500a0deb79026b162ee7b89914228 (patch) | |
tree | 74f32a9014ffb0ebc3b8268af15fa6a569f65063 /doc/development/code_comments.md | |
parent | f904971e5fefff92b0cc467a4dc4483ba68b6e6c (diff) | |
download | gitlab-ce-75fd8a40b99500a0deb79026b162ee7b89914228.tar.gz |
Add code comments development guide
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Diffstat (limited to 'doc/development/code_comments.md')
-rw-r--r-- | doc/development/code_comments.md | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/doc/development/code_comments.md b/doc/development/code_comments.md new file mode 100644 index 00000000000..36962eb46d4 --- /dev/null +++ b/doc/development/code_comments.md @@ -0,0 +1,14 @@ +# Code comments + +Whenever you add comment to the code that is expected to be addressed at any time +in future, please create a technical debt issue for it. Then put a link to it +to the code comment you've created. This will allow other developers to quickly +check if a comment is still relevant and what needs to be done to address it. + +Examples: + +```rb +# Deprecated scope until code_owner column has been migrated to rule_type. +# To be removed with https://gitlab.com/gitlab-org/gitlab-ee/issues/11834. +scope :code_owner, -> { where(code_owner: true).or(where(rule_type: :code_owner)) } +``` |