Skip to content

node_version_prompt should work without NVM #1946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jul 13, 2025
Merged

node_version_prompt should work without NVM #1946

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b935ddd
`node_version_prompt` should work without NVM
kigster Sep 13, 2021
1a96dd1
lib/theme: use `_command_exists()`
gaelicWizard Feb 24, 2022
4125123
Merge pull request #3 from gaelicWizard/theme/nodejs-version
kigster Feb 24, 2022
f7c3256
Update themes/base.theme.bash
kigster Feb 24, 2022
78d19e7
Fix node and ruby colors
kigster Mar 24, 2022
289a9d9
Merge branch 'master' into kig/nodejs-version
kigster Mar 24, 2022
7a0b127
Merge branch 'master' into kig/nodejs-version
seefood Jul 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 21 additions & 0 deletions themes/base.theme.bash
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -394,6 +394,27 @@
fi
}

function node_command_version_prompt() {
local node_version
if _command_exists node; then
node_version="$(node --version 2> /dev/null)"
if [[ -n ${node_version} ]]; then
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This check would give a false positive when using version management shims that are common with tools like asdf

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you clarify? What do you mean by a false positive?

if node --version returns a non-empty output, doesn't that mean that this is the current node version? If so, why not show it in the prompt?

echo -e "${NVM_THEME_PROMPT_PREFIX}${node_version}${NVM_THEME_PROMPT_SUFFIX}"
fi
fi
}

function node_version_prompt() {
local node_version

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
node_version="$(nvm_version_prompt)"

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
if [[ -z "${node_version}" ]]; then

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
node_version="$(node_command_version_prompt)"

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
fi
Comment on lines +410 to +412

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
if [[ -n "${node_version}" ]]; then

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
echo -e "${node_version}"

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
fi
Comment on lines +413 to +415

Check warning

Code scanning / shellcheck

SC2317 Warning

Command appears to be unreachable. Check usage (or ignore if invoked indirectly).
}

function nvm_version_prompt() {
local node
if _is_function nvm; then
Expand Down
4 changes: 2 additions & 2 deletions themes/powerline-multiline/powerline-multiline.theme.bash
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,13 +45,13 @@ NVM_THEME_PROMPT_SUFFIX=""
NODE_THEME_PROMPT_PREFIX=""
NODE_THEME_PROMPT_SUFFIX=""
NODE_CHAR=${POWERLINE_NODE_CHAR:="❲n❳ "}
NODE_THEME_PROMPT_COLOR=${POWERLINE_NODE_COLOR:=22}
NODE_THEME_PROMPT_COLOR=${POWERLINE_NODE_COLOR:=28}

RVM_THEME_PROMPT_PREFIX=""
RVM_THEME_PROMPT_SUFFIX=""
RBENV_THEME_PROMPT_PREFIX=""
RBENV_THEME_PROMPT_SUFFIX=""
RUBY_THEME_PROMPT_COLOR=${POWERLINE_RUBY_COLOR:=161}
RUBY_THEME_PROMPT_COLOR=${POWERLINE_RUBY_COLOR:=1}
RUBY_CHAR=${POWERLINE_RUBY_CHAR:="❲r❳ "}

TERRAFORM_THEME_PROMPT_COLOR=${POWERLINE_TERRAFORM_COLOR:=161}
Expand Down
Loading