Skip to content

Commit 6898015

Browse files
author
Tod Beardsley
committed
Just skip if info is suppressed.
1 parent 81a9808 commit 6898015

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

tools/msftidy.rb

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
require 'time'
1212

1313
CHECK_OLD_RUBIES = !!ENV['MSF_CHECK_OLD_RUBIES']
14-
CHECK_RUBOCOP = !!ENV['MSF_CHECK_RUBOCOP']
1514
SUPPRESS_INFO_MESSAGES = !!ENV['MSF_SUPPRESS_INFO_MESSAGES']
1615

1716
if CHECK_OLD_RUBIES
@@ -321,8 +320,10 @@ def check_extname
321320
end
322321
end
323322

323+
# Explicitly skip this check if we're suppressing info messages
324+
# anyway, since it takes a fair amount of time per module to perform.
324325
def check_rubocop
325-
return true unless CHECK_RUBOCOP
326+
return true if SUPPRESS_INFO_MESSAGES
326327
out = %x{rubocop -n #{@full_filepath}}
327328
ret = $?
328329
info("Fails to pass Rubocop Ruby style guidelines (run 'rubocop #{@full_filepath}' to see violations)") unless ret.exitstatus == 0

0 commit comments

Comments
 (0)