-
-
Notifications
You must be signed in to change notification settings - Fork 45
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
Join forces with Bash IDE #920
Comments
@skovhus I also think that providing a single, better maintained extension would provide a better experience for end-users. However, just to be clear, we're talking about deprecating vscode-shellcheck in favor of Bash IDE. Right? As you mentioned, at the moment Bash IDE lacks some features that are present in VS Code ShellCheck. The ones I can think of:
@timonwong what's your opinion about this? |
Thanks for writing this up. There could like be some features that is harder or not possible to do with an language server protocol (LSP) server compared to a vscode extension. I'm not 100% on this, but I'm unsure on how to provide grammar for
As bash language server has a much wider scope and is applicable to all editors following the LSP protocol – eventually deprecating vscode-shellcheck could be natural step. Similar idea presented in #114. But I also see a world where the two projects could co-exist and share fundamentals (like ShellCheck binaries) and features. As mentioned the fact that vscode-shellcheck is a vscode only extension does give some flexibility compare to bash-language-server that follows the language server protocol. As for the features lacking then I believe your list it correct, although we do support alternatives shells based on the shebang and file extension of the file. |
It should be possible as part of the VS Code client only, at least in worst case. Anyway, grammar for this file that such a meaningful feature.
Well, that's great then! |
Originally posted by @skovhus in #916 (comment)
Originally posted by @skovhus in bash-lsp/bash-language-server#711 (comment)
Thus, I'm opening this thread for discussing this. The Bash IDE repository can be found at https://github.com/bash-lsp/bash-language-server.
The text was updated successfully, but these errors were encountered: