Improve error discovery when the LSP process cannot be started #142
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
When the
bash-language-servercannot be started or found the user only sees a message that the process could not be found instead of seeing the actual reason for the failure.This message got me very confused when I installed the extension because I made sure I had the
bash-language-serverinstalled and could find it in myPATHbut I thought VS Code somehow fails to find it. The actual problem was that I upgraded to Node.js 12 and the process failed to start as it was not compiled against the correct Node.js version. I only discovered that when I actually tried to start the server.This PR improves the discovery of the failure by showing a more generic error message and guiding users to the DevTools console where the actual error is logged.