Fix for review

Author: Shougo

autoload/denops/cache.vim

@@ -5,6 +5,7 @@ let s:job = v:null
 function! denops#cache#update(...) abort
   const l:options = extend(#{ reload: v:false }, a:0 ? a:1 : {})
   const l:plugins = denops#_internal#plugin#list()
+  let l:failures = []
 
   if !l:options.reload
     echomsg '[denops] Update cache of the following files. Call `denops#cache#update(#{reload: v:true})` to forcibly update.'
@@ -21,7 +22,7 @@ function! denops#cache#update(...) abort
 
     let s:job = denops#_internal#job#start(l:args, #{
           \ on_stderr: funcref('s:on_stderr'),
-          \ on_exit: funcref('s:on_exit'),
+          \ on_exit: funcref('s:on_exit', [l:failures, l:entryfile]),
           \ env: #{
           \   NO_COLOR: 1,
           \   DENO_NO_PROMPT: 1,
@@ -30,7 +31,16 @@ function! denops#cache#update(...) abort
     call denops#_internal#wait#for(60 * 1000, { -> s:job is# v:null }, 100)
   endfor
 
-  echomsg '[denops] Deno cache is updated.'
+  if empty(l:failures)
+    echomsg '[denops] Deno cache is updated.'
+  else
+    echohl WarningMsg
+    echomsg printf('[denops] Deno cache finished with errors: %d failure(s).', len(l:failures))
+    for l:f in l:failures
+      echomsg printf('[denops] failed: %s', l:f)
+    endfor
+    echohl None
+  endif
 endfunction
 
 function! s:on_stderr(job, data, event) abort
@@ -41,6 +51,9 @@ function! s:on_stderr(job, data, event) abort
   echohl None
 endfunction
 
-function! s:on_exit(job, status, event) abort
+function! s:on_exit(failures, entryfile, job, status, event) abort
+  if a:status != 0
+    call add(a:failures, a:entryfile)
+  endif
   let s:job = v:null
 endfunction