From e5c599ed9db21bf914e44be4c639f614ff29e272 Mon Sep 17 00:00:00 2001
From: Richard Schneeman <rschneeman@salesforce.com>
Date: Fri, 6 Oct 2023 12:55:10 -0500
Subject: [PATCH] Update build output

---
 buildpacks/ruby/src/steps/detect_rake_tasks.rs   | 11 ++++-------
 buildpacks/ruby/src/steps/get_default_process.rs |  8 ++++----
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/buildpacks/ruby/src/steps/detect_rake_tasks.rs b/buildpacks/ruby/src/steps/detect_rake_tasks.rs
index cec3fd8f..b7758814 100644
--- a/buildpacks/ruby/src/steps/detect_rake_tasks.rs
+++ b/buildpacks/ruby/src/steps/detect_rake_tasks.rs
@@ -29,7 +29,7 @@ pub(crate) fn detect_rake_tasks(
     ) {
         RakeStatus::MissingRakeGem => {
             log_step(format!(
-                "Cannot run rake tasks {}",
+                "Skipping rake tasks {}",
                 fmt::details(format!("no {rake} gem in {gemfile}"))
             ));
 
@@ -42,7 +42,7 @@ pub(crate) fn detect_rake_tasks(
         }
         RakeStatus::MissingRakefile => {
             log_step(format!(
-                "Cannot run rake tasks {}",
+                "Skipping rake tasks {}",
                 fmt::details(format!("no {rakefile}"))
             ));
             log_step(format!("{HELP} Add {rakefile} to your project to enable",));
@@ -66,11 +66,8 @@ pub(crate) fn detect_rake_tasks(
         }
         RakeStatus::Ready(path) => {
             log_step(format!(
-                "Rake detected {}",
-                fmt::details(format!(
-                    "{rake} gem found, {rakefile} found ad {path}",
-                    path = fmt::value(path.to_string_lossy())
-                ))
+                "Detected rake ({rake} gem found, {rakefile} found at {path})",
+                path = fmt::value(path.to_string_lossy())
             ));
 
             let rake_detect = RakeDetect::from_rake_command(logger, env, true)
diff --git a/buildpacks/ruby/src/steps/get_default_process.rs b/buildpacks/ruby/src/steps/get_default_process.rs
index 3f368195..aad981d1 100644
--- a/buildpacks/ruby/src/steps/get_default_process.rs
+++ b/buildpacks/ruby/src/steps/get_default_process.rs
@@ -20,27 +20,27 @@ pub(crate) fn get_default_process(
     let railties = fmt::value("railties");
     match detect_web(gem_list, &context.app_dir) {
         WebProcess::Rails => {
-            log_step(format!("Detected rails app ({rails} gem)"));
+            log_step(format!("Detected rails app ({rails} gem found)"));
 
             Some(default_rails())
         }
         WebProcess::RackWithConfigRU => {
             log_step(format!(
-                "Detected rack app ({rack} gem and {config_ru} at root of application)"
+                "Detected rack app ({rack} gem found and {config_ru} at root of application)"
             ));
 
             Some(default_rack())
         }
         WebProcess::RackMissingConfigRu => {
             log_step(format!(
-                "Skipping default web process (detected {rack} gem but missing {config_ru} file"
+                "Skipping default web process ({rack} gem found but missing {config_ru} file)"
             ));
 
             None
         }
         WebProcess::Missing => {
             log_step(format!(
-                "Skipping default web process (no web gems detected: {rails}, {railties}, {rack}"
+                "Skipping default web process ({rails}, {railties}, and {rack} not found)"
             ));
 
             None