.rubocop.yml in rutl-0.1.4 vs .rubocop.yml in rutl-0.2.0

- old
+ new

@@ -1,5 +1,8 @@ +# The exclusions here and in source code are intentional decisions to not +# follow Rubocop's advice. Anything that should be fiex but isn't is in +# .rubocop_todo.yml inherit_from: .rubocop_todo.yml require: rubocop-rspec # drewcoo-cops has to go after rubocop-rspec because otherwise it diables # drewcoo-cops. @@ -12,5 +15,30 @@ AllCops: DefaultFormatter: progress DisplayCopNames: true DisplayStyleGuide: true ExtraDetails: true + +# Metrics/BlockLength, Metrics/MethodLength +# don't apply to the spec files. They naturally have long swaths of code. +# And the gemspec is special. +Metrics/BlockLength: + Exclude: + - 'rutl.gemspec' + - 'spec/*_spec.rb' +Metrics/MethodLength: + Exclude: + - 'spec/*_spec.rb' + +# TODO: Change this setting? Ideally, I should force braces on the *Page files. +# But should think about this first. +Style/BracesAroundHashParameters: + Exclude: + - 'spec/pages/**/*' + +# NullDriverPageElement plays dirty tricks with @@variables to simulate +# longer-lived data sources. +# And BasePage does @@loaded_pages. +Style/ClassVars: + Exclude: + - 'lib/rutl/driver/null_driver_page_element.rb' + - 'lib/rutl/base_page.rb'