From 427f48aa8dd8d7b95b4da96d507b40b7050b62c3 Mon Sep 17 00:00:00 2001 From: Gerry Date: Fri, 10 Jul 2020 17:41:25 +0200 Subject: [PATCH] fix(nodejs): remove line break from version (#1464) A recent refactor of modules to use format strings accidentally got rid of the `trim()` on the NodeJS version string. This just adds it back so that the prompt doesn't include an unnecessary line break when showing that module. --- src/modules/nodejs.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/modules/nodejs.rs b/src/modules/nodejs.rs index f9088f52..9f3dcfc8 100644 --- a/src/modules/nodejs.rs +++ b/src/modules/nodejs.rs @@ -30,6 +30,7 @@ pub fn module<'a>(context: &'a Context) -> Option> { let mut module = context.new_module("nodejs"); let config = NodejsConfig::try_load(module.config); + let nodejs_version = utils::exec_cmd("node", &["--version"])?.stdout; let parsed = StringFormatter::new(config.format).and_then(|formatter| { formatter .map_meta(|var, _| match var { @@ -41,7 +42,7 @@ pub fn module<'a>(context: &'a Context) -> Option> { _ => None, }) .map(|variable| match variable { - "version" => Some(Ok(utils::exec_cmd("node", &["--version"])?.stdout)), + "version" => Some(Ok(nodejs_version.trim())), _ => None, }) .parse(None)