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?