diff options
| author | Robert Speicher <rspeicher@gmail.com> | 2016-05-23 13:10:42 -0400 |
|---|---|---|
| committer | Robert Speicher <rspeicher@gmail.com> | 2016-05-23 13:10:42 -0400 |
| commit | 51c167eeb75f429098b7128f9d9f33a6eec4145f (patch) | |
| tree | 87566fa8d3193d0c709c788ab74de9ce15100b75 | |
| parent | bc806831e818b6e1ec2d1ae81221126f03c01f11 (diff) | |
| download | gitlab-ce-rs-range-include.tar.gz | |
Enable Performance/RangeInclude cop and fix single offensers-range-include
| -rw-r--r-- | .rubocop.yml | 3 | ||||
| -rw-r--r-- | lib/gitlab/bitbucket_import/client.rb | 2 |
2 files changed, 2 insertions, 3 deletions
diff --git a/.rubocop.yml b/.rubocop.yml index 0946ef5d848..26929e03456 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -957,10 +957,9 @@ Performance/EndWith: Performance/LstripRstrip: Enabled: true -# TODO: Enable RangeInclude Cop. # Use `Range#cover?` instead of `Range#include?`. Performance/RangeInclude: - Enabled: false + Enabled: true # TODO: Enable RedundantBlockCall Cop. # Use `yield` instead of `block.call`. diff --git a/lib/gitlab/bitbucket_import/client.rb b/lib/gitlab/bitbucket_import/client.rb index 9b83292ef33..8d1ad62fae0 100644 --- a/lib/gitlab/bitbucket_import/client.rb +++ b/lib/gitlab/bitbucket_import/client.rb @@ -121,7 +121,7 @@ module Gitlab def get(url) response = api.get(url) - raise Unauthorized if (400..499).include?(response.code.to_i) + raise Unauthorized if (400..499).cover?(response.code.to_i) response end |