refactor: satisfy newer Rubocop version

Author: ffloyd

.rubocop.yml

@@ -1,10 +1,12 @@
+inherit_from: .rubocop_todo.yml
+
 require:
   - rubocop-rspec
   - rubocop-performance
   - rubocop-md
 
 AllCops:
-  TargetRubyVersion: 2.7
+  TargetRubyVersion: 2.5
   NewCops: enable
 
 Style/HashEachMethods:

.rubocop_todo.yml

@@ -0,0 +1,27 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config --auto-gen-only-exclude`
+# on 2020-08-22 07:49:05 UTC using RuboCop version 0.89.1.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 27
+#
+# *** NON-GENERATED COMMENT ***
+# I'm not sure about enabling this cop
+# * using super may has performance impact
+# * but for some cases it still can be useful
+# Research needed
+Lint/MissingSuper:
+  Enabled: false
+
+# Offense count: 5
+# Configuration parameters: AllowSubject, Max.
+#
+# *** NON-GENERATED COMMENT ***
+# Might be it's impossible to satisfy this cop or significant refactoring of these tests needed
+RSpec/MultipleMemoizedHelpers:
+  Exclude:
+    - 'spec/flows/flow/node_spec.rb'
+    - 'spec/flows/util/inheritable_singleton_vars/isolation_strategy_spec.rb'

bin/benchmark_cli/examples/.rubocop.yml

@@ -17,3 +17,6 @@ Lint/UselessAssignment:
 
 Metrics/MethodLength:
   Enabled: false
+
+Metrics/AbcSize:
+  Enabled: false

flows.gemspec

@@ -21,6 +21,8 @@ Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
+  spec.required_ruby_version = '>= 2.5'
+
   # This library has no production dependencies.
   # So, it will not block you from updating any dependencies in your project.
   # So, don't add production dependencies.

lib/flows/plugin/output_contract/dsl.rb

@@ -39,7 +39,7 @@ def failure_with(status, &contract_block)
         # Disables contract check and transformation for current class and children.
         #
         # @param enable [Boolean] if true - contracts are disabled
-        def skip_output_contract(enable = true)
+        def skip_output_contract(enable: true)
           @skip_output_contract_flag = enable
         end
       end

spec/flows/railway_spec.rb

@@ -29,7 +29,7 @@ def sum(arg1:, arg2:)
         end
 
         def square(sum:)
-          ok(square: sum * sum)
+          ok(square: sum**2)
         end
       end
     end