Merge pull request #14 from boost/bump-rubocop-version

Bump rubocop version

Author: hapiben

.travis.yml

@@ -4,4 +4,4 @@ language: ruby
 cache: bundler
 rvm:
  - 2.5.1
-before_install: gem install bundler -v 2.0.1
+before_install: gem install bundler -v 2.0.2

Gemfile.lock

@@ -3,58 +3,58 @@ PATH
  specs:
  boost-styles (0.1.0)
  haml_lint (~> 0.33)
- rubocop (~> 0.77)
- rubocop-performance (~> 1.5.1)
- rubocop-rails (~> 2.4.0)
+ rubocop (~> 0.78)
+ rubocop-performance (~> 1.5)
+ rubocop-rails (~> 2.4)
  rubocop-rspec (~> 1.37)
 
 GEM
  remote: https://rubygems.org/
  specs:
- actioncable (5.2.3)
- actionpack (= 5.2.3)
+ actioncable (5.2.4.1)
+ actionpack (= 5.2.4.1)
  nio4r (~> 2.0)
  websocket-driver (>= 0.6.1)
- actionmailer (5.2.3)
- actionpack (= 5.2.3)
- actionview (= 5.2.3)
- activejob (= 5.2.3)
+ actionmailer (5.2.4.1)
+ actionpack (= 5.2.4.1)
+ actionview (= 5.2.4.1)
+ activejob (= 5.2.4.1)
  mail (~> 2.5, >= 2.5.4)
  rails-dom-testing (~> 2.0)
- actionpack (5.2.3)
- actionview (= 5.2.3)
- activesupport (= 5.2.3)
- rack (~> 2.0)
+ actionpack (5.2.4.1)
+ actionview (= 5.2.4.1)
+ activesupport (= 5.2.4.1)
+ rack (~> 2.0, >= 2.0.8)
  rack-test (>= 0.6.3)
  rails-dom-testing (~> 2.0)
  rails-html-sanitizer (~> 1.0, >= 1.0.2)
- actionview (5.2.3)
- activesupport (= 5.2.3)
+ actionview (5.2.4.1)
+ activesupport (= 5.2.4.1)
  builder (~> 3.1)
  erubi (~> 1.4)
  rails-dom-testing (~> 2.0)
  rails-html-sanitizer (~> 1.0, >= 1.0.3)
- activejob (5.2.3)
- activesupport (= 5.2.3)
+ activejob (5.2.4.1)
+ activesupport (= 5.2.4.1)
  globalid (>= 0.3.6)
- activemodel (5.2.3)
- activesupport (= 5.2.3)
- activerecord (5.2.3)
- activemodel (= 5.2.3)
- activesupport (= 5.2.3)
+ activemodel (5.2.4.1)
+ activesupport (= 5.2.4.1)
+ activerecord (5.2.4.1)
+ activemodel (= 5.2.4.1)
+ activesupport (= 5.2.4.1)
  arel (>= 9.0)
- activestorage (5.2.3)
- actionpack (= 5.2.3)
- activerecord (= 5.2.3)
+ activestorage (5.2.4.1)
+ actionpack (= 5.2.4.1)
+ activerecord (= 5.2.4.1)
  marcel (~> 0.3.1)
- activesupport (5.2.3)
+ activesupport (5.2.4.1)
  concurrent-ruby (~> 1.0, >= 1.0.2)
  i18n (>= 0.7, < 2)
  minitest (~> 5.1)
  tzinfo (~> 1.1)
  arel (9.0.0)
  ast (2.4.0)
- builder (3.2.3)
+ builder (3.2.4)
  concurrent-ruby (1.1.5)
  crass (1.0.5)
  diff-lcs (1.3)
@@ -64,15 +64,15 @@ GEM
  haml (5.1.2)
  temple (>= 0.8.0)
  tilt
- haml_lint (0.34.0)
+ haml_lint (0.34.1)
  haml (>= 4.0, < 5.2)
  rainbow
  rubocop (>= 0.50.0)
  sysexits (~> 1.1)
  i18n (1.7.0)
  concurrent-ruby (~> 1.0)
  jaro_winkler (1.5.4)
- loofah (2.3.1)
+ loofah (2.4.0)
  crass (~> 1.0.2)
  nokogiri (>= 1.5.9)
  mail (2.7.1)
@@ -83,37 +83,37 @@ GEM
  mimemagic (0.3.3)
  mini_mime (1.0.2)
  mini_portile2 (2.4.0)
- minitest (5.12.2)
+ minitest (5.13.0)
  nio4r (2.5.2)
- nokogiri (1.10.4)
+ nokogiri (1.10.7)
  mini_portile2 (~> 2.4.0)
  parallel (1.19.1)
- parser (2.6.5.0)
+ parser (2.7.0.1)
  ast (~> 2.4.0)
- rack (2.0.7)
+ rack (2.0.8)
  rack-test (1.1.0)
  rack (>= 1.0, < 3)
- rails (5.2.3)
- actioncable (= 5.2.3)
- actionmailer (= 5.2.3)
- actionpack (= 5.2.3)
- actionview (= 5.2.3)
- activejob (= 5.2.3)
- activemodel (= 5.2.3)
- activerecord (= 5.2.3)
- activestorage (= 5.2.3)
- activesupport (= 5.2.3)
+ rails (5.2.4.1)
+ actioncable (= 5.2.4.1)
+ actionmailer (= 5.2.4.1)
+ actionpack (= 5.2.4.1)
+ actionview (= 5.2.4.1)
+ activejob (= 5.2.4.1)
+ activemodel (= 5.2.4.1)
+ activerecord (= 5.2.4.1)
+ activestorage (= 5.2.4.1)
+ activesupport (= 5.2.4.1)
  bundler (>= 1.3.0)
- railties (= 5.2.3)
+ railties (= 5.2.4.1)
  sprockets-rails (>= 2.0.0)
  rails-dom-testing (2.0.3)
  activesupport (>= 4.2.0)
  nokogiri (>= 1.6)
  rails-html-sanitizer (1.3.0)
  loofah (~> 2.3)
- railties (5.2.3)
- actionpack (= 5.2.3)
- activesupport (= 5.2.3)
+ railties (5.2.4.1)
+ actionpack (= 5.2.4.1)
+ activesupport (= 5.2.4.1)
  method_source
  rake (>= 0.8.7)
  thor (>= 0.19.0, < 2.0)
@@ -123,28 +123,28 @@ GEM
  rspec-core (~> 3.9.0)
  rspec-expectations (~> 3.9.0)
  rspec-mocks (~> 3.9.0)
- rspec-core (3.9.0)
- rspec-support (~> 3.9.0)
+ rspec-core (3.9.1)
+ rspec-support (~> 3.9.1)
  rspec-expectations (3.9.0)
  diff-lcs (>= 1.2.0, < 2.0)
  rspec-support (~> 3.9.0)
- rspec-mocks (3.9.0)
+ rspec-mocks (3.9.1)
  diff-lcs (>= 1.2.0, < 2.0)
  rspec-support (~> 3.9.0)
- rspec-support (3.9.0)
- rubocop (0.77.0)
+ rspec-support (3.9.2)
+ rubocop (0.78.0)
  jaro_winkler (~> 1.5.1)
  parallel (~> 1.10)
  parser (>= 2.6)
  rainbow (>= 2.2.2, < 4.0)
  ruby-progressbar (~> 1.7)
  unicode-display_width (>= 1.4.0, < 1.7)
- rubocop-performance (1.5.1)
+ rubocop-performance (1.5.2)
  rubocop (>= 0.71.0)
- rubocop-rails (2.4.0)
+ rubocop-rails (2.4.1)
  rack (>= 1.1)
  rubocop (>= 0.72.0)
- rubocop-rspec (1.37.0)
+ rubocop-rspec (1.37.1)
  rubocop (>= 0.68.1)
  ruby-progressbar (1.10.1)
  sprockets (4.0.0)
@@ -156,10 +156,10 @@ GEM
  sprockets (>= 3.0.0)
  sysexits (1.2.0)
  temple (0.8.2)
- thor (0.20.3)
+ thor (1.0.1)
  thread_safe (0.3.6)
  tilt (2.0.10)
- tzinfo (1.2.5)
+ tzinfo (1.2.6)
  thread_safe (~> 0.1)
  unicode-display_width (1.6.0)
  websocket-driver (0.7.1)

boost-styles.gemspec

@@ -43,8 +43,8 @@ Gem::Specification.new do |spec|
  spec.add_development_dependency 'rspec', '~> 3.0'
 
  spec.add_dependency 'haml_lint', '~> 0.33'
- spec.add_dependency 'rubocop', '~> 0.77'
- spec.add_dependency 'rubocop-performance', '~> 1.5.1'
+ spec.add_dependency 'rubocop', '~> 0.78'
+ spec.add_dependency 'rubocop-performance', '~> 1.5'
+ spec.add_dependency 'rubocop-rails', '~> 2.4'
  spec.add_dependency 'rubocop-rspec', '~> 1.37'
- spec.add_dependency 'rubocop-rails', '~> 2.4.0'
 end

rubocop_default.yml

@@ -7,3 +7,7 @@ inherit_from:
  - rubocop_rails.yml
  - rubocop_performance.yml
  - rubocop_rspec.yml
+ 
+ - rubocop_layout.yml
+ - rubocop_metrics.yml
+ - rubocop_style.yml

rubocop_layout.yml

@@ -0,0 +1,5 @@
+# Method definitions after `private` or `protected` isolated calls need one
+# extra level of indentation.
+Layout/IndentationConsistency:
+ Enabled: true
+ EnforcedStyle: indented_internal_methods

rubocop_metrics.yml

@@ -0,0 +1,46 @@
+# A calculated magnitude based on number of assignments,
+# branches, and conditions.
+Metrics/AbcSize:
+ Enabled: true
+ Max: 54.28
+
+# This cop checks if the length of a block exceeds some maximum value.
+Metrics/BlockLength:
+ Enabled: false
+
+# Avoid excessive block nesting.
+Metrics/BlockNesting:
+ Enabled: true
+ Max: 4
+
+# Avoid classes longer than 100 lines of code.
+Metrics/ClassLength:
+ Enabled: false
+
+# A complexity metric that is strongly correlated to the number
+# of test cases needed to validate a method.
+Metrics/CyclomaticComplexity:
+ Enabled: true
+ Max: 13
+
+# Limit lines to 80 characters.
+Metrics/LineLength:
+ Enabled: false
+
+# Avoid methods longer than 10 lines of code.
+Metrics/MethodLength:
+ Enabled: false
+
+# Avoid modules longer than 100 lines of code.
+Metrics/ModuleLength:
+ Enabled: false
+
+# Avoid parameter lists longer than three or four parameters.
+Metrics/ParameterLists:
+ Enabled: true
+ Max: 8
+
+# A complexity metric geared towards measuring complexity for a human reader.
+Metrics/PerceivedComplexity:
+ Enabled: true
+ Max: 14

rubocop_style.yml

@@ -0,0 +1,12 @@
+Style/FrozenStringLiteralComment:
+ Enabled: true
+ EnforcedStyle: always
+ Exclude:
+ - 'actionview/test/**/*.builder'
+ - 'actionview/test/**/*.ruby'
+ - 'actionpack/test/**/*.builder'
+ - 'actionpack/test/**/*.ruby'
+ - 'activestorage/db/migrate/**/*.rb'
+
+Style/Documentation:
+ Enabled: false