Done todo

Author: mtsmfm

.rubocop.yml

@@ -1,8 +1,45 @@
 inherit_from:
   - https://github.com/onk/onkcop/raw/v0.51.0.0/config/rubocop.yml
-  - .rubocop_todo.yml
 
 AllCops:
   TargetRubyVersion: 2.2
   Exclude:
     - 'bin/**/*'
+
+# TODO: Remove this config after droping Ruby 2.2 support
+Layout/IndentHeredoc:
+  Include:
+    - 'test/**/*'
+
+Lint/AssignmentInCondition:
+  Enabled: false
+
+Lint/HandleExceptions:
+  Enabled: false
+
+Naming/HeredocDelimiterNaming:
+  Enabled: false
+
+Style/BlockDelimiters:
+  Enabled: false
+
+Style/MultilineBlockChain:
+  Enabled: false
+
+Style/Semicolon:
+  Enabled: false
+
+Metrics/AbcSize:
+  Max: 50
+
+Metrics/BlockLength:
+  Max: 30
+
+Metrics/MethodLength:
+  Max: 40
+
+Metrics/ModuleLength:
+  Enabled: false
+
+Metrics/LineLength:
+  Max: 200

.rubocop_todo.yml

@@ -37,9 +37,7 @@ def notify(method:, params: {})
 
         method = request[:method].to_sym
 
-        _, subscriber = subscribers.find { |k, _|
-          k === method
-        }
+        subscriber = subscribers[method]
 
         if subscriber
           keys = subscriber.parameters.map(&:last)

lib/language_server.rb

@@ -78,7 +78,7 @@ def on_stmts_add(*args)
         end
 
         def on_assign(left, right)
-          result.constants << left if Constant === left
+          result.constants << left if left.instance_of?(Constant)
 
           left.value = right if left.respond_to?(:value) # TODO: remove this condition
           left