Bug - explainshell services has no /api path, so arg tips doesn't work
palansher opened this issue · comments
Hello!
Thanks you for nice tool!
bash-language-server/server/src/analyser.ts
Line 636 in afd0d82
If replace
const url = `${endpoint}/api/explain?${searchParams}`
to
const url = `${endpoint}/explain?${searchParams}`
then will be working requests like this:
https://explainshell.com/api/explain?cmd=grep+-v+%22current%22
to https://explainshell.com or custom buit https://github.com/idank/explainshell
The reason of that: there is no /api path supported on known explainshell services.
Currently all requests to /api return 404 error
Thanks, they must have changed their URL. Are you up for doing a PR?
There is some documentation and a security warning here:
https://github.com/bash-lsp/bash-language-server/tree/main/vscode-client#configuration
Are you up for doing a PR
Hello! Maybe I can. If you kindly explain what is 'PR' , you mentioned :)
The Pull Request? I am a junior coder, But I will try.
Yes, I meant a pull request. :)