Skip to content

Commit 50e12fe

Browse files
committed
Perform all format_specializations_check and only fail after
1 parent d26c7e1 commit 50e12fe

File tree

1 file changed

+11
-4
lines changed

1 file changed

+11
-4
lines changed

tool/jt.rb

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3045,10 +3045,17 @@ def final_classes
30453045
end
30463046

30473047
def format_specializations_check
3048-
abort 'Some Specializations did not use the protected visibility.' if format_specializations_visibility
3049-
abort 'Some Specializations were not properly formatted.' if format_specializations_arguments
3050-
abort 'There were extra blank lines around imports.' if Formatting.format_imports
3051-
abort 'There were classes which should be marked as final but were not.' if final_classes
3048+
any_failed = false
3049+
[
3050+
[format_specializations_visibility, 'Some Specializations did not use the protected visibility.'],
3051+
[format_specializations_arguments, 'Some Specializations were not properly formatted.'],
3052+
[Formatting.format_imports, 'There were extra blank lines around imports.'],
3053+
[final_classes, 'There were classes which should be marked as final but were not.'],
3054+
].each do |changed, error_message|
3055+
any_failed ||= changed
3056+
$stderr.puts error_message if changed
3057+
end
3058+
abort if any_failed
30523059
end
30533060

30543061
def check_generated_files

0 commit comments

Comments
 (0)