Replace IfNode mixin with node extension

This change replaces the use of a cop mixin for access to logic related to `if` nodes, and instead decorates said nodes with the appropriate module upon initialization.
yakryder · Jan 5, 2017 · 219dfe1 · 219dfe1
1 parent c36c4e0
commit 219dfe1
Show file tree

Hide file tree

Showing 44 changed files with 924 additions and 404 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -13,7 +13,7 @@ Metrics/AbcSize:
 # Offense count: 32
 # Configuration parameters: CountComments.
 Metrics/ClassLength:
- Max: 182
+ Max: 183
 
 # Offense count: 29
 Metrics/CyclomaticComplexity:

diff --git a/lib/rubocop.rb b/lib/rubocop.rb
@@ -20,6 +20,7 @@
 require 'rubocop/node_pattern'
 require 'rubocop/string_interpreter'
 require 'rubocop/ast_node/sexp'
+require 'rubocop/ast_node/node_extensions/if_node'
 require 'rubocop/ast_node'
 require 'rubocop/ast_node/builder'
 require 'rubocop/ast_node/traversal'
@@ -67,7 +68,6 @@
 require 'rubocop/cop/mixin/frozen_string_literal'
 require 'rubocop/cop/mixin/hash_node'
 require 'rubocop/cop/mixin/hash_alignment'
-require 'rubocop/cop/mixin/if_node'
 require 'rubocop/cop/mixin/integer_node'
 require 'rubocop/cop/mixin/on_method_def'
 require 'rubocop/cop/mixin/match_range'