diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-12-17 11:59:07 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-12-17 11:59:07 +0000 |
commit | 8b573c94895dc0ac0e1d9d59cf3e8745e8b539ca (patch) | |
tree | 544930fb309b30317ae9797a9683768705d664c4 /doc/development/module_with_instance_variables.md | |
parent | 4b1de649d0168371549608993deac953eb692019 (diff) | |
download | gitlab-ce-8b573c94895dc0ac0e1d9d59cf3e8745e8b539ca.tar.gz |
Add latest changes from gitlab-org/gitlab@13-7-stable-eev13.7.0-rc42
Diffstat (limited to 'doc/development/module_with_instance_variables.md')
-rw-r--r-- | doc/development/module_with_instance_variables.md | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/doc/development/module_with_instance_variables.md b/doc/development/module_with_instance_variables.md index 80e926f800c..75575105178 100644 --- a/doc/development/module_with_instance_variables.md +++ b/doc/development/module_with_instance_variables.md @@ -1,7 +1,7 @@ --- stage: none group: unassigned -info: To determine the technical writer assigned to the Stage/Group associated with this page, see https://about.gitlab.com/handbook/engineering/ux/technical-writing/#designated-technical-writers +info: To determine the technical writer assigned to the Stage/Group associated with this page, see https://about.gitlab.com/handbook/engineering/ux/technical-writing/#assignments --- # Modules with instance variables could be considered harmful @@ -121,7 +121,7 @@ module Gitlab end ``` -Now the cop won't complain. Here's a bad example which we could rewrite: +Now the cop doesn't complain. Here's a bad example which we could rewrite: ``` ruby module SpamCheckService @@ -213,14 +213,14 @@ module M end ``` -Note that you need to enable it at some point, otherwise everything below -won't be checked. +Note that you need to enable it at some point, otherwise nothing below +that point is checked. ## Things we might need to ignore right now Because of the way Rails helpers and mailers work, we might not be able to avoid the use of instance variables there. For those cases, we could ignore -them at the moment. At least we're not going to share those modules with +them at the moment. Those modules are not shared with other random objects, so they're still somewhat isolated. ## Instance variables in views |