Merge pull request #7 from quirinecker/feature/2-when-ascidoctor-reveal-js-is-missing-it-should-say-so-instead-of-saying-asciidoctor-itself-is-missing

When ascidoctor reveal js is missing it should say so instead of saying asciidoctor itself is missing

Closes #2
This commit is contained in:
2025-11-10 13:49:15 +01:00
committed by GitHub

View File

@@ -12,9 +12,13 @@ fn exec_command(command: &mut process::Command) -> Result<(), String> {
} }
} else { } else {
println!("{}", result.unwrap_err()); println!("{}", result.unwrap_err());
return Err(
"asciidoctor not installed. For more information run docki health!".to_string(), let binary_name = command.get_program().to_str().unwrap_or("Something is");
);
return Err(format!(
"{} not installed. For more information run docki health!",
binary_name
));
} }
} }
@@ -31,11 +35,11 @@ fn asciidoctor_docs(in_path: &str, out_path: &str) -> process::Command {
fn asciidoctor_slides(in_path: &str, out_path: &str, offline_reveal: bool) -> process::Command { fn asciidoctor_slides(in_path: &str, out_path: &str, offline_reveal: bool) -> process::Command {
let mut command = process::Command::new(format!("asciidoctor-revealjs")); let mut command = process::Command::new(format!("asciidoctor-revealjs"));
let out_dir = parent_path(out_path); let out_dir = parent_path(out_path);
let revealjs_path = if offline_reveal { let revealjs_path = if offline_reveal {
path_between(out_dir.to_string(), "./dist/slides/revealjs".to_string()) path_between(out_dir.to_string(), "./dist/slides/revealjs".to_string())
} else { } else {
"https://cdn.jsdelivr.net/npm/reveal.js@5.2.1".to_string() "https://cdn.jsdelivr.net/npm/reveal.js@5.2.1".to_string()
}; };
command command
.arg(format!("{in_path}")) .arg(format!("{in_path}"))