Disable the Rails/RakeEnvironment cop

We're not writing application code in which it's ok to assume that all tasks should load the environment first.
solidusio · Dec 18, 2023 · 542a111 · 542a111
1 parent 44140a0
commit 542a111
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 10 deletions.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -345,3 +345,8 @@ Lint/MissingSuper:
 Rails/FindEach:
   Exclude:
     - 'db/migrate/**/*'
+
+# Since we're writing library code we can't assume that
+# tasks should load the rails environment loaded.
+Rails/RakeEnvironment:
+  Enabled: false
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -459,16 +459,6 @@ Rails/Present:
     - "core/lib/spree/core/search/base.rb"
     - "core/spec/models/spree/stock/availability_validator_spec.rb"
 
-# Offense count: 8
-# This cop supports unsafe autocorrection (--autocorrect-all).
-# Configuration parameters: Include.
-# Include: **/Rakefile, **/*.rake
-Rails/RakeEnvironment:
-  Exclude:
-    - "Rakefile"
-    - "backend/Rakefile"
-    - "core/Rakefile"
-
 # Offense count: 8
 # This cop supports safe autocorrection (--autocorrect).
 Rails/RedundantForeignKey: