Errors should be printed regardless of whether the verbose options are set. <p dir

errors not printed unless verbose options set about neoformat HOT 3 CLOSED

sbdchd commented on July 28, 2024

errors not printed unless verbose options set

from neoformat.

Comments (3)

sbdchd commented on July 28, 2024

v:shell_error just returns the exit code of the last command, which I don't think is very necessary for the user to see. It can be useful for debugging which is why I use log.

The following diff prints:

Neoformat: 126
Neoformat: attempted all formatters for current filetype
Press ENTER or type command to continue

vs the normal

Neoformat: attempted all formatters for current filetype

diff --git a/autoload/neoformat.vim b/autoload/neoformat.vim
index 7157f97..1643433 100644
--- a/autoload/neoformat.vim
+++ b/autoload/neoformat.vim
@@ -111,7 +111,7 @@ function! s:neoformat(bang, user_input, start_line, end_line) abort
                 return neoformat#utils#msg('no change necessary with ' . cmd.name)
             endif
         else
-            call neoformat#utils#log(v:shell_error)
+            call neoformat#utils#msg(v:shell_error)
             call neoformat#utils#log('trying next formatter')
         endif
     endfor
diff --git a/autoload/neoformat/formatters/python.vim b/autoload/neoformat/formatters/python.vim
index 97958e4..0829705 100644
--- a/autoload/neoformat/formatters/python.vim
+++ b/autoload/neoformat/formatters/python.vim
@@ -4,7 +4,7 @@ endfunction
 
 function! neoformat#formatters#python#yapf() abort
     return {
-                \ 'exe': 'yapf',
+                \ 'exe': 'apf',
                 \ 'stdin': 1
                 \ }
 endfunction