[Fix #546] Exclude app/models by default for Rails/ContentTag

Fixes #546 This PR excludes `app/models` by default for `Rails/ContentTag`. Because it prevents false positives for `tag` calls to `has_one: tag`. No helpers are used in normal models.
rubocop · Sep 14, 2021 · 47f7993 · 47f7993
1 parent 41245e0
commit 47f7993
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/changelog/change_exclude_models_by_default_for_rails_content_tag.md b/changelog/change_exclude_models_by_default_for_rails_content_tag.md
@@ -0,0 +1 @@
+* [#546](https://github.com/rubocop/rubocop-rails/issues/546): Exclude `app/models` by default for `Rails/ContentTag`. ([@koic][])
diff --git a/config/default.yml b/config/default.yml
@@ -189,6 +189,9 @@ Rails/ContentTag:
   Enabled: true
   VersionAdded: '2.6'
   VersionChanged: '2.12'
+  # This `Exclude` config prevents false positives for `tag` calls to `has_one: tag`. No helpers are used in normal models.
+  Exclude:
+    - app/models/**/*.rb
 
 Rails/CreateTableWithTimestamps:
   Description: >-
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		* [#546](https://github.com/rubocop/rubocop-rails/issues/546): Exclude `app/models` by default for `Rails/ContentTag`. ([@koic][])