Allow visibility scoped node matchers

rubocop · Apr 2, 2017 · ef729df · ef729df
1 parent ef9607c
commit ef729df
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -19,12 +19,17 @@ Style/FrozenStringLiteralComment:
 Style/IndentHeredoc:
   EnforcedStyle: powerpack
 
-Metrics/BlockLength:
+Lint/AmbiguousBlockAssociation:
   Exclude:
-    - 'Rakefile'
-    - '**/*.rake'
     - 'spec/**/*.rb'
 
-AmbiguousBlockAssociation:
+Lint/UselessAccessModifier:
+  MethodCreatingMethods:
+    - 'def_matcher'
+    - 'def_node_matcher'
+
+Metrics/BlockLength:
   Exclude:
+    - 'Rakefile'
+    - '**/*.rake'
     - 'spec/**/*.rb'
diff --git a/lib/rubocop/ast/node/send_node.rb b/lib/rubocop/ast/node/send_node.rb
@@ -184,7 +184,7 @@ def node_parts
         [*self]
       end
 
-      private # rubocop:disable Lint/UselessAccessModifier
+      private
 
       def_matcher :macro_scope?, <<-PATTERN
         {^({class module} ...)
-Original file line number
+Diff line change
@@ Expand Up / @@ -184,7 +184,7 @@ def node_parts @@
             [*self]
           end
-          private # rubocop:disable Lint/UselessAccessModifier
+          private
           def_matcher :macro_scope?, <<-PATTERN
             {^({class module} ...)
@@ Expand Down @@