common_rubocop_rails.yml in radius-spec-0.1.1 vs common_rubocop_rails.yml in radius-spec-0.2.0

- old
+ new

@@ -9,13 +9,15 @@ Enabled: true AllCops: Exclude: - 'bin/puma' + - 'bin/pumactl' - 'bin/rails' - 'bin/shoryuken' - 'bin/sidekiq' + - 'bin/sidekiqctl' - 'bin/spring' - 'db/schema.rb' - 'db/migrate/**/*' # Rails project's are not concerned with API docs normally @@ -33,9 +35,20 @@ # URISchemes: http, https Metrics/LineLength: IgnoredPatterns: - '\A# fk_rails_' - '\A# index_' + +# Ignore subclass parent for one off benchmarks +# +# Benchmarks are generally meant to be small and targeted. They often have +# custom model declarations which help direct the focus of the benchmarks. +# These one-off models exist outside of the main Rails app. We don't want this +# cop to run for them because including `ApplicationRecord` is unnecessary in +# these cases and just adds noise to the setup. +Rails/ApplicationRecord: + Exclude: + - 'benchmarks/**/*' # As Rails defaults to creating timestamps you need to go out of your way to # explicitly have them removed from the table. This cop is almost always a # false negative so we're disabling it. #