aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorXu Cheng2015-02-17 18:02:05 +0800
committerMike McQuaid2015-02-17 10:30:42 +0000
commit86cd5d2f249295bef02186098265ec9996f9f421 (patch)
tree0ca02a52fe5f68436b2382b300a9c231b3fbf48b
parentf204e2e60879a3ebf335c0a260a4a84c125dde0d (diff)
downloadbrew-86cd5d2f249295bef02186098265ec9996f9f421.tar.bz2
test-bot: fail fast in check_results
Closes Homebrew/homebrew#36893. Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
-rw-r--r--Library/Homebrew/cmd/test-bot.rb8
1 files changed, 3 insertions, 5 deletions
diff --git a/Library/Homebrew/cmd/test-bot.rb b/Library/Homebrew/cmd/test-bot.rb
index 9a1478764..2617aee87 100644
--- a/Library/Homebrew/cmd/test-bot.rb
+++ b/Library/Homebrew/cmd/test-bot.rb
@@ -549,15 +549,13 @@ module Homebrew
end
def check_results
- status = :passed
- steps.each do |step|
+ steps.all? do |step|
case step.status
- when :passed then next
+ when :passed then true
when :running then raise
- when :failed then status = :failed
+ when :failed then false
end
end
- status == :passed
end
def formulae