From eacd788cf56fee586901f4252ae7492aaf007b3f Mon Sep 17 00:00:00 2001 From: Billy Vong Date: Thu, 16 Nov 2017 14:43:50 -0800 Subject: [PATCH] :checkhealth: fix check for npm and yarn (#7569) 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 --- runtime/autoload/health/provider.vim | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/runtime/autoload/health/provider.vim b/runtime/autoload/health/provider.vim index 806a9d043b..0201ed8062 100644 --- a/runtime/autoload/health/provider.vim +++ b/runtime/autoload/health/provider.vim @@ -496,7 +496,7 @@ function! s:check_node() abort return endif - if !executable('node') || !executable('npm') || !executable('yarn') + if !executable('node') || !executable('npm') call health#report_warn( \ '`node` and `npm` must be in $PATH.', \ ['Install Node.js and verify that `node` and `npm` commands work.'])