diff options
| author | Robert Speicher <robert@gitlab.com> | 2016-07-29 15:58:09 +0000 |
|---|---|---|
| committer | Robert Speicher <robert@gitlab.com> | 2016-07-29 15:58:09 +0000 |
| commit | 34c083a184b98372b3b28a661d5cf41e0f2d8259 (patch) | |
| tree | 54832269f981fa1fc0480ad1c2d448b7fc476718 /app/helpers/diff_helper.rb | |
| parent | 4284724de4ac86595dfa09cc1f8301770d667db7 (diff) | |
| parent | d6f669774481b160c2d963b56309ab6262216c42 (diff) | |
| download | gitlab-ce-34c083a184b98372b3b28a661d5cf41e0f2d8259.tar.gz | |
Merge branch 'rubocop/enable-access-modifiers-cops' into 'master'
Enable Rubocop cops that check access modifiers
## What does this MR do?
This MR enables Rubocop cops that detect methods that should be restricted but are the part of public API because of access modifiers used improperly.
This also fixes existing offenses.
## Why was this MR needed?
Some method in our codebase are public instead of being private because it is sometimes difficult to get it right without static analysis.
## What are the relevant issue numbers?
See #17478
Closes #17372
See merge request !5014
Diffstat (limited to 'app/helpers/diff_helper.rb')
| -rw-r--r-- | app/helpers/diff_helper.rb | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/app/helpers/diff_helper.rb b/app/helpers/diff_helper.rb index 4c031942793..f35e2f6ddcd 100644 --- a/app/helpers/diff_helper.rb +++ b/app/helpers/diff_helper.rb @@ -144,8 +144,6 @@ module DiffHelper toggle_whitespace_link(url, options) end - private - def hide_whitespace? params[:w] == '1' end |