From b67f40bdf7448718bd401e8f9217ad60375b2f47 Mon Sep 17 00:00:00 2001 From: Donal McBreen Date: Thu, 2 May 2024 12:38:20 +0100 Subject: [PATCH] Warn on missing builder We are going to try to create a builder if one is missing, so let's warn rather than report it as an error. --- lib/kamal/cli/build.rb | 2 +- test/cli/build_test.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/kamal/cli/build.rb b/lib/kamal/cli/build.rb index 8eae4dd7..a43cee2e 100644 --- a/lib/kamal/cli/build.rb +++ b/lib/kamal/cli/build.rb @@ -30,7 +30,7 @@ class Kamal::Cli::Build < Kamal::Cli::Base end rescue SSHKit::Command::Failed => e if e.message =~ /(no builder)|(no such file or directory)/ - error "Missing compatible builder, so creating a new one first" + warn "Missing compatible builder, so creating a new one first" if cli.create KAMAL.with_verbosity(:debug) { execute *KAMAL.builder.push } diff --git a/test/cli/build_test.rb b/test/cli/build_test.rb index 5c3e3680..ca11dc1a 100644 --- a/test/cli/build_test.rb +++ b/test/cli/build_test.rb @@ -34,7 +34,7 @@ class CliBuildTest < CliTestCase .returns(true) run_command("push").tap do |output| - assert_match /Missing compatible builder, so creating a new one first/, output + assert_match /WARN Missing compatible builder, so creating a new one first/, output end end