Merge pull request #307 from koic/set_disabled_by_default_for_rails_p…

…luck_id [Fix #301] Set disabled by default for `Rails/PluckId`
rubocop · Jul 23, 2020 · 7adb747 · 7adb747
2 parents fa6915f + 9fc3f67
commit 7adb747
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@
 
 * [#297](https://github.com/rubocop-hq/rubocop-rails/pull/297): Handle an upstream Ruby issue where the DidYouMean module is not available, which would break the `Rails/UnknownEnv` cop. ([@taylorthurlow][])
 
+### Changes
+
+* [#301](https://github.com/rubocop-hq/rubocop-rails/issues/301): Set disalbed by default for `Rails/PluckId`. ([@koic][])
+
 ## 2.7.0 (2020-07-21)
 
 ### New features

diff --git a/config/default.yml b/config/default.yml
@@ -417,7 +417,7 @@ Rails/Pluck:
 Rails/PluckId:
   Description: 'Use `ids` instead of `pluck(:id)` or `pluck(primary_key)`.'
   StyleGuide: 'https://rails.rubystyle.guide/#ids'
-  Enabled: 'pending'
+  Enabled: false
   Safe: false
   VersionAdded: '2.7'
 

diff --git a/docs/modules/ROOT/pages/cops_rails.adoc b/docs/modules/ROOT/pages/cops_rails.adoc
@@ -2408,7 +2408,7 @@ Post.published.pluck(:title)
 |===
 | Enabled by default | Safe | Supports autocorrection | VersionAdded | VersionChanged
 
-| Pending
+| Disabled
 | No
 | Yes (Unsafe)
 | 2.7