kevin/neovim
1
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2024-12-29 14:41:06 -07:00
Code Issues Packages Projects Releases Wiki Activity

:checkhealth: fix check for npm and yarn (#7569)

Browse Source 
Fix bug that checked for npm AND yarn, where we wanted npm OR yarn.
But since we call `npm` exclusively, and it's highly unlikely you have
yarn installed without npm, let's just remove the yarn check altogether.

Addresses https://github.com/neovim/node-client/issues/41
This commit is contained in:
Billy Vong 2017-11-16 14:43:50 -08:00 committed by Justin M. Keyes
parent 59b0d9f62d
commit eacd788cf5
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
runtime/autoload/health/provider.vim
View File

@ -496,7 +496,7 @@ function! s:check_node() abort
return return
endif endif
if !executable('node') || !executable('npm') || !executable('yarn') if !executable('node') || !executable('npm')
call health#report_warn( call health#report_warn(
\ '`node` and `npm` must be in $PATH.', \ '`node` and `npm` must be in $PATH.',
\ ['Install Node.js and verify that `node` and `npm` commands work.']) \ ['Install Node.js and verify that `node` and `npm` commands work.'])