Merge pull request #2 from todb-r7/rubocop-is-optional

Rubocop checks are optional and info only

Author: jhart-r7

tools/msftidy.rb

@@ -320,10 +320,13 @@ def check_extname
     end
   end
 
+  # Explicitly skip this check if we're suppressing info messages
+  # anyway, since it takes a fair amount of time per module to perform.
   def check_rubocop
+    return true if SUPPRESS_INFO_MESSAGES
     out = %x{rubocop -n #{@full_filepath}}
     ret = $?
-    error("Fails to pass Rubocop Ruby style guidelines (run 'rubocop #{@full_filepath}' to see violations)") unless ret.exitstatus == 0
+    info("Fails to pass Rubocop Ruby style guidelines (run 'rubocop #{@full_filepath}' to see violations)") unless ret.exitstatus == 0
   end
 
   def check_old_rubies