common_rubocop_rails.yml in radius-spec-0.6.0 vs common_rubocop_rails.yml in radius-spec-0.7.0
- old
+ new
@@ -1,16 +1,14 @@
+require: rubocop-rails
+
inherit_mode:
merge:
- Exclude
- IgnoredPatterns
inherit_from: common_rubocop.yml
-# Enable additional Rails cops
-Rails:
- Enabled: true
-
AllCops:
Exclude:
- 'bin/puma'
- 'bin/pumactl'
- 'bin/rails'
@@ -141,9 +139,16 @@
# do want a lookup table.
#
# Configuration parameters: Include.
# Include: app/models/**/*.rb
Rails/HasAndBelongsToMany:
+ Enabled: false
+
+# We find the combo `:only` and `:if` readable. While the `:except` and `:if`
+# combo is easier to read as a combined proc. As a team we are fine with
+# handling this in PR reviews, until such time which Rubocop provides an option
+# for us to configure this.
+Rails/IgnoredSkipActionFilterOption:
Enabled: false
# The ActiveSupport monkey patches for `present?` are nearly all defined as:
#
# !blank?