Run internal investigation using master only. Fix disabling

rubocop · Jun 19, 2020 · 2817ae4 · 2817ae4
1 parent 8e595c3
commit 2817ae4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/rubocop.yml b/.github/workflows/rubocop.yml
@@ -66,7 +66,7 @@ jobs:
         if: matrix.coverage != true
         run:  bundle exec rake spec
       - name: internal_investigation
-        if: "matrix.os != 'windows' && matrix.coverage != true"
+        if: "matrix.os != 'windows' && matrix.coverage != true && matrix.rubocop == 'master'"
         run:  bundle exec rake internal_investigation
   rubocop_specs:
     name: >-

diff --git a/lib/rubocop/ast/node.rb b/lib/rubocop/ast/node.rb
@@ -505,7 +505,7 @@ def guard_clause?
       # So, does the return value of this node matter? If we changed it to
       # `(...; nil)`, might that affect anything?
       #
-      # rubocop:disable Metrics/MethodLength, Metrics/CyclomaticComplexity
+      # rubocop:disable Metrics/MethodLength
       def value_used?
         # Be conservative and return true if we're not sure.
         return false if parent.nil?
@@ -527,7 +527,7 @@ def value_used?
           true
         end
       end
-      # rubocop:enable Metrics/MethodLength, Metrics/CyclomaticComplexity
+      # rubocop:enable Metrics/MethodLength
 
       # Some expressions are evaluated for their value, some for their side
       # effects, and some for both.