From aa79ec3ca7ae2571f4d180015c0bd8195056c705 Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Fri, 6 Dec 2019 13:27:41 -0800 Subject: Simplify static-analysis output generation The buffering of the output may be causing issues, so let's disable it for now. --- scripts/static-analysis | 61 +++++++------------------------------------------ 1 file changed, 8 insertions(+), 53 deletions(-) (limited to 'scripts/static-analysis') diff --git a/scripts/static-analysis b/scripts/static-analysis index 642c50ec0a8..0972fe081f5 100755 --- a/scripts/static-analysis +++ b/scripts/static-analysis @@ -4,27 +4,6 @@ require_relative '../lib/gitlab/popen' require_relative '../lib/gitlab/popen/runner' -def emit_warnings(static_analysis) - static_analysis.warned_results.each do |result| - puts - puts "**** #{result.cmd.join(' ')} had the following warning(s):" - puts - puts result.stderr - puts - end -end - -def emit_errors(static_analysis) - static_analysis.failed_results.each do |result| - puts - puts "**** #{result.cmd.join(' ')} failed with the following error(s):" - puts - puts result.stdout - puts result.stderr - puts - end -end - tasks = [ %w[bin/rake lint:all], %w[bundle exec license_finder], @@ -36,37 +15,13 @@ tasks = [ %w[scripts/lint-rugged] ] -static_analysis = Gitlab::Popen::Runner.new - -static_analysis.run(tasks) do |cmd, &run| - puts - puts "$ #{cmd.join(' ')}" - - result = run.call +tasks.each do |task| + cmd = task.join(' ') + puts "$ #{cmd}" + result = system(cmd) - puts "==> Finished in #{result.duration} seconds" - puts -end - -puts -puts '===================================================' -puts -puts - -if static_analysis.all_success_and_clean? - puts 'All static analyses passed successfully.' -elsif static_analysis.all_success? - puts 'All static analyses passed successfully, but we have warnings:' - puts - - emit_warnings(static_analysis) - - exit 2 -else - puts 'Some static analyses failed:' - - emit_warnings(static_analysis) - emit_errors(static_analysis) - - exit 1 + unless result + puts "#{cmd} failed!" + exit 1 + end end -- cgit v1.2.1