[Fix #197] Mark Performance/ArraySemiInfiniteRangeSlice cop as unsafe

This cop is unsafe for string slices because string does not have `#take` and `#drop` methods Closes #197
rubocop · Nov 23, 2020 · f2ddd5b · f2ddd5b
1 parent fd9e060
commit f2ddd5b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## master (unreleased)
 
+### Changes
+
+* [#197](https://github.com/rubocop-hq/rubocop-performance/pull/197): Mark `Performance/ArraySemiInfiniteRangeSlice` cop as unsafe. ([@tejasbubane][])
+
 ## 1.9.0 (2020-11-17)
 
 ### New features

diff --git a/config/default.yml b/config/default.yml
@@ -10,6 +10,8 @@ Performance/AncestorsInclude:
 Performance/ArraySemiInfiniteRangeSlice:
   Description: 'Identifies places where slicing arrays with semi-infinite ranges can be replaced by `Array#take` and `Array#drop`.'
   Enabled: pending
+  # Unsafe for string slices because string does not have `#take` and `#drop` methods
+  Safe: false
   VersionAdded: '1.9'
 
 Performance/BigDecimalWithNumericArgument:

diff --git a/docs/modules/ROOT/pages/cops_performance.adoc b/docs/modules/ROOT/pages/cops_performance.adoc
@@ -38,8 +38,8 @@ NOTE: Required Ruby version: 2.7
 | Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
 
 | Pending
-| Yes
-| Yes
+| No
+| Yes (Unsafe)
 | 1.9
 | -
 |===