-
Notifications
You must be signed in to change notification settings - Fork 84
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.
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
Consider setting suppress_stderr to v:true by default #213
Comments
Hiding the output makes sense for the servers that emit a bunch of noise. It wouldn't make sense for the servers which use stderr to report real problems and then stop working. It's less confusing to see that an error happened than to stop getting language features with no indication of why. The linked python server issue is a good example. These look like serious errors and not a choice to emit logging to stderr. The fact that (some?) features continue to work after the errors may not be a solid signal that it is OK to ignore. I do think it probably makes sense to deprecate the global config in favor of per-server config. I'm hoping that we might move in a direction where more authors publish plugins like vim-lsc-dart to centralize language config, and then those authors would be in the best position to know how to treat stderr. We still need to pick a default in that case... Are there particular servers that you are interested in which do log noise to stderr? Do you think if we made it possible to configure this on a per-server basis you'd want to publish a plugin that does so? |
I think the per-language configs we have now are mostly enough. The
server I use the most is
|
FWIW LSP defines ways for servers to send messages that should be user-visible, but writing to stderr isn't one of them. My favorite behavior would be:
|
That sounds like a really nice behavior. I'll take a shot at implementing that. |
Some servers seem to log a lot of unnecessary stuff to
stderr
, most of it not needed by the users. Setting it tov:true
by default in a future release it at least worth considering, in my opinion.The text was updated successfully, but these errors were encountered: