Skip to content

Commit 771e750

Browse files
authored
Land rapid7#19342, Update msftidy logging for older modules
2 parents c4fd2fa + ed5c959 commit 771e750

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

spec/tools/dev/rubocop_runner_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,8 @@ def patch_io_read_encoding
7272
expect(@status).to be_zero
7373
end
7474

75-
it 'contains no warnings' do
76-
expect(@stdout).to be_empty
75+
it 'contains a status message' do
76+
expect(@stdout).to match /Rubocop not required for older modules skipping/
7777
end
7878
end
7979
end

tools/dev/msftidy.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ class RuboCopRunner
5555
# @return [Integer] RuboCop::CLI status code
5656
def run(full_filepath, options = {})
5757
unless requires_rubocop?(full_filepath)
58+
puts "#{full_filepath} - [*] Rubocop not required for older modules skipping. If making a large update - run rubocop #{"rubocop -a #{full_filepath}".yellow} and verify all issues are resolved"
5859
return RuboCop::CLI::STATUS_SUCCESS
5960
end
6061

0 commit comments

Comments
 (0)