Skip to content

Commit

Permalink
Update build output
Browse files Browse the repository at this point in the history
  • Loading branch information
schneems committed Oct 6, 2023
1 parent a97dec6 commit e5c599e
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 11 deletions.
11 changes: 4 additions & 7 deletions buildpacks/ruby/src/steps/detect_rake_tasks.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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}"))
));

Expand All @@ -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",));
Expand All @@ -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)
Expand Down
8 changes: 4 additions & 4 deletions buildpacks/ruby/src/steps/get_default_process.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit e5c599e

Please sign in to comment.