.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'