Revert "Bump RuboCop to 0.47.1 (#89)"

This reverts commit dbaeb02.

Author: maxjacobson

Dockerfile

@@ -4,8 +4,7 @@ WORKDIR /usr/src/app
 COPY Gemfile /usr/src/app/
 COPY Gemfile.lock /usr/src/app/
 
-RUN gem update --system && \
-    gem install bundler && \
+RUN gem install bundler && \
     bundle install -j 4 && \
     rm -fr /usr/share/ri
 

Gemfile

@@ -5,7 +5,7 @@ source 'https://rubygems.org'
 gem "activesupport", require: false
 gem "parser", "~> 2.3.3.1"
 gem "pry", require: false
-gem "rubocop", "~> 0.47.1", require: false
+gem "rubocop", "~> 0.45", require: false
 gem "rubocop-migrations", require: false
 gem "rubocop-rspec", require: false
 gem "safe_yaml"

Gemfile.lock

@@ -20,7 +20,7 @@ GEM
       coderay (~> 1.1.0)
       method_source (~> 0.8.1)
       slop (~> 3.4)
-    rainbow (2.2.1)
+    rainbow (2.1.0)
     rake (12.0.0)
     rspec (3.5.0)
       rspec-core (~> 3.5.0)
@@ -35,8 +35,8 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.5.0)
     rspec-support (3.5.0)
-    rubocop (0.47.1)
-      parser (>= 2.3.3.1, < 3.0)
+    rubocop (0.46.0)
+      parser (>= 2.3.1.1, < 3.0)
       powerpack (~> 0.1)
       rainbow (>= 1.99.1, < 3.0)
       ruby-progressbar (~> 1.7)
@@ -51,7 +51,7 @@ GEM
     thread_safe (0.3.5)
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
-    unicode-display_width (1.1.3)
+    unicode-display_width (1.1.2)
 
 PLATFORMS
   ruby
@@ -62,10 +62,10 @@ DEPENDENCIES
   pry
   rake
   rspec
-  rubocop (~> 0.47.1)
+  rubocop (~> 0.45)
   rubocop-migrations
   rubocop-rspec
   safe_yaml
 
 BUNDLED WITH
-   1.14.4
+   1.13.6

config/contents/lint/ambiguous_operator.md

@@ -2,16 +2,11 @@ This cop checks for ambiguous operators in the first argument of a
 method invocation without parentheses.
 
 ### Example:
-
-    # bad
+    array = [1, 2, 3]
 
     # The `*` is interpreted as a splat operator but it could possibly be
-    # a `*` method invocation (i.e. `do_something.*(some_array)`).
-    do_something *some_array
-
-### Example:
-
-    # good
+    # a `*` method invocation (i.e. `do_something.*(array)`).
+    do_something *array
 
     # With parentheses, there's no ambiguity.
-    do_something(*some_array)
+    do_something(*array)

config/contents/lint/ambiguous_regexp_literal.md

@@ -2,17 +2,10 @@ This cop checks for ambiguous regexp literals in the first argument of
 a method invocation without parentheses.
 
 ### Example:
-
-    # bad
-
     # This is interpreted as a method invocation with a regexp literal,
     # but it could possibly be `/` method invocations.
     # (i.e. `do_something./(pattern)./(i)`)
     do_something /pattern/i
 
-### Example:
-
-    # good
-
     # With parentheses, there's no ambiguity.
     do_something(/pattern/i)

config/contents/lint/assignment_in_condition.md

@@ -1,8 +1,8 @@
 This cop checks whether the end keywords are aligned properly for do
 end blocks.
 
-Three modes are supported through the `EnforcedStyleAlignWith`
-configuration parameter:
+Three modes are supported through the `AlignWith` configuration
+parameter:
 
 `start_of_block` : the `end` shall be aligned with the
 start of the line where the `do` appeared.
@@ -15,40 +15,18 @@ location. The autofixer will default to `start_of_line`.
 
 ### Example:
 
-    # bad
-
-    foo.bar
-       .each do
-         baz
-           end
-
-### Example:
-
-    # EnforcedStyleAlignWith: either (default)
-
-    # good
-
+    # either
     variable = lambda do |i|
       i
     end
 
-### Example:
-
-    # EnforcedStyleAlignWith: start_of_block
-
-    # good
-
+    # start_of_block
     foo.bar
       .each do
          baz
        end
 
-### Example:
-
-    # EnforcedStyleAlignWith: start_of_line
-
-    # good
-
+    # start_of_line
     foo.bar
       .each do
          baz

config/contents/lint/block_alignment.md

@@ -4,41 +4,27 @@ arguments and optional ordinal arguments.
 This cop mirrors a warning produced by MRI since 2.2.
 
 ### Example:
-
     # bad
-
     def bake(pie: pie)
       pie.heat_up
     end
 
-### Example:
-
     # good
-
     def bake(pie:)
       pie.refrigerate
     end
 
-### Example:
-
     # good
-
     def bake(pie: self.pie)
       pie.feed_to(user)
     end
 
-### Example:
-
     # bad
-
     def cook(dry_ingredients = dry_ingredients)
       dry_ingredients.reduce(&:+)
     end
 
-### Example:
-
     # good
-
     def cook(dry_ingredients = self.dry_ingredients)
       dry_ingredients.combine
     end

config/contents/lint/circular_argument_reference.md

@@ -3,17 +3,7 @@ if/while/until.
 
 ### Example:
 
-    # bad
-
     if
       some_condition
       do_something
-    end
-
-### Example:
-
-    # good
-
-    if some_condition
-      do_something
     end

config/contents/lint/condition_position.md

@@ -1,33 +1,13 @@
 This cop checks whether the end keywords of method definitions are
 aligned properly.
 
-Two modes are supported through the EnforcedStyleAlignWith configuration
+Two modes are supported through the AlignWith configuration
 parameter. If it's set to `start_of_line` (which is the default), the
 `end` shall be aligned with the start of the line where the `def`
 keyword is. If it's set to `def`, the `end` shall be aligned with the
 `def` keyword.
 
 ### Example:
 
-    # bad
-
-    private def foo
-                end
-
-### Example:
-
-    # EnforcedStyleAlignWith: start_of_line (default)
-
-    # good
-
-    private def foo
-    end
-
-### Example:
-
-    # EnforcedStyleAlignWith: def
-
-    # good
-
     private def foo
-            end
+    end

config/contents/lint/debugger.md

@@ -3,22 +3,10 @@ used in case 'when' expressions.
 
 ### Example:
 
-    # bad
-
-    case x
-    when 'first'
-      do_something
-    when 'first'
-      do_something_else
-    end
-
-### Example:
-
-    # good
-
-    case x
-    when 'first
-      do_something
-    when 'second'
-      do_something_else
-    end
+     # bad
+     case x
+     when 'first'
+       do_something
+     when 'first'
+       do_something_else
+     end

config/contents/lint/def_end_alignment.md

@@ -2,25 +2,11 @@ This cop checks for duplicated instance (or singleton) method
 definitions.
 
 ### Example:
-
     # bad
-
     def duplicated
       1
     end
 
     def duplicated
       2
-    end
-
-### Example:
-
-    # good
-
-    def duplicated
-      1
-    end
-
-    def other_duplicated
-      2
     end

config/contents/lint/deprecated_class_methods.md

@@ -3,13 +3,4 @@ This cop checks for duplicated keys in hash literals.
 This cop mirrors a warning in Ruby 2.2.
 
 ### Example:
-
-    # bad
-
-    hash = { food: 'apple', food: 'orange' }
-
-### Example:
-
-    # good
-
-    hash = { food: 'apple', other_food: 'orange' }
+    hash = { food: 'apple', food: 'orange' }