X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/e731dd71a77ad4ad1e9db22535e89444bb98e5d9..9d8ca8be11b6a529eb352064b1ea9a3975c4e065:/.rubocop.yml
diff --git a/.rubocop.yml b/.rubocop.yml
index cbd62a052..60d144544 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -1,11 +1,16 @@
inherit_from: .rubocop_todo.yml
require:
+ - rubocop-capybara
+ - rubocop-factory_bot
+ - rubocop-minitest
- rubocop-performance
- rubocop-rails
+ - rubocop-rake
AllCops:
- TargetRubyVersion: 2.5
+ TargetRubyVersion: 3.1
+ NewCops: enable
Exclude:
- 'vendor/**/*'
@@ -29,21 +34,27 @@ Metrics/ClassLength:
Exclude:
- 'test/**/*'
+Metrics/ModuleLength:
+ Max: 150
+
Naming/FileName:
Exclude:
- 'script/deliver-message'
- - 'script/locale/reload-languages'
- 'script/update-spam-blocks'
-Naming/UncommunicativeMethodParamName:
+Naming/MethodParameterName:
Enabled: false
-Rails/ApplicationRecord:
+# This conflicts with Strong Migrations, which can't check `change_table`
+Rails/BulkChangeTable:
Enabled: false
Rails/CreateTableWithTimestamps:
Enabled: false
+Rails/FindBy:
+ IgnoreWhereFirst: false
+
Rails/FindEach:
Enabled: false
@@ -53,19 +64,20 @@ Rails/HasManyOrHasOneDependent:
Rails/HttpPositionalArguments:
Enabled: false
-Rails/InverseOf:
+Rails/ReflectionClassName:
Enabled: false
Rails/SkipsModelValidations:
Exclude:
- 'db/migrate/*.rb'
- - 'app/controllers/users_controller.rb'
+ - 'app/controllers/users/lists_controller.rb'
-Style/BracesAroundHashParameters:
- EnforcedStyle: context_dependent
+Style/Documentation:
+ Enabled: false
Style/FormatStringToken:
- EnforcedStyle: template
+ Exclude:
+ - 'config/routes.rb'
Style/IfInsideElse:
Enabled: false
@@ -73,12 +85,21 @@ Style/IfInsideElse:
Style/GuardClause:
Enabled: false
+Style/HashEachMethods:
+ Enabled: true
+
Style/HashSyntax:
EnforcedStyle: hash_rockets
Exclude:
- 'lib/tasks/testing.rake'
- 'config/initializers/wrap_parameters.rb'
+Style/HashTransformKeys:
+ Enabled: true
+
+Style/HashTransformValues:
+ Enabled: true
+
Style/MixinUsage:
Exclude:
- 'bin/setup'