[Fix #384] Mark unsafe for Rails/NegateInclude

Fixes #384

This PR marks unsafe for `Rails/NegateInclude` because false positive
will occur for a receiver object that do not have `exclude?` method.
(e.g. `IPAddr`)

CHANGELOG.md

@@ -12,6 +12,7 @@
 
 * [#371](https://github.com/rubocop-hq/rubocop-rails/pull/371): Fix an infinite loop error for `Rails/ActiveRecordCallbacksOrder` when callbacks have inline comments. ([@fatkodima][])
 * [#364](https://github.com/rubocop-hq/rubocop-rails/pull/364): Fix a problem that `Rails/UniqueValidationWithoutIndex` doesn't work in classes defined with compact style. ([@sinsoku][])
+* [#384](https://github.com/rubocop-hq/rubocop-rails/issues/384): Mark unsafe for `Rails/NegateInclude`. ([@koic][])
 
 ### Changes
 

config/default.yml

@@ -400,7 +400,9 @@ Rails/NegateInclude:
   Description: 'Prefer `collection.exclude?(obj)` over `!collection.include?(obj)`.'
   StyleGuide: 'https://rails.rubystyle.guide#exclude'
   Enabled: 'pending'
+  Safe: false
   VersionAdded: '2.7'
+  VersionChanged: '2.9'
 
 Rails/NotNullColumn:
   Description: 'Do not add a NOT NULL column without a default value.'

docs/modules/ROOT/pages/cops_rails.adoc

@@ -2314,15 +2314,18 @@ match 'photos/:id', to: 'photos#show', via: :all
 | Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
 
 | Pending
-| Yes
-| Yes
+| No
+| Yes (Unsafe)
 | 2.7
-| -
+| 2.9
 |===
 
 This cop enforces the use of `collection.exclude?(obj)`
 over `!collection.include?(obj)`.
 
+It is marked as unsafe by default because false positive will occur for
+a receiver object that do not have `exclude?` method. (e.g. `IPAddr`)
+
 === Examples
 
 [source,ruby]

lib/rubocop/cop/rails/negate_include.rb

@@ -6,6 +6,9 @@ module Rails
       # This cop enforces the use of `collection.exclude?(obj)`
       # over `!collection.include?(obj)`.
       #
+      # It is marked as unsafe by default because false positive will occur for
+      # a receiver object that do not have `exclude?` method. (e.g. `IPAddr`)
+      #
       # @example
       #   # bad
       #   !array.include?(2)