Skip to content
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

Cooperate with ShellCheck extension #19

Open
KlfJoat opened this issue Feb 22, 2024 · 2 comments
Open

Cooperate with ShellCheck extension #19

KlfJoat opened this issue Feb 22, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@KlfJoat
Copy link

KlfJoat commented Feb 22, 2024

Hi. I really appreciate this extension and how much easier it makes Bats test writing. Thank you so much for creating and maintaining it!

I use another VSC extension, timonwong.shellcheck, to lint my Bash code with ShellCheck. Both ShellCheck and that extension support linting Bats files.

However, the ShellCheck extension only lints files that have the "ShellScript" language identifier. And your wonderful extension creates a new configured language identifier, "Bats (Bash Automated Testing System)". So timonwong.shellcheck won't run on Bats files while your extension is enabled.

When I want to lint my Bats files, I disable your extension, manually set the individual file as a ShellScript, then fix issues. I have 45 Bats files in this project so far, and it's barely an eighth completed.

I opened an issue for this with them, thinking a fix was on their side. But they pointed out that they have an API for other extensions to interface with. https://github.com/vscode-shellcheck/vscode-shellcheck/blob/master/doc%2FAPI.md

It seems like the way for Bats language support to be enabled and linted by ShellCheck is if you could please hook into their API. I'd help if I could, but I tend to write bash scripts, not javascript extensions so I'm not sure how much help I could be beyond beta testing.

Thanks!

@jetmartin jetmartin added the enhancement New feature or request label Feb 22, 2024
@jetmartin
Copy link
Collaborator

Hi,

Thank you for your great feedback and proposed enhancement !
What you said make sens. With all the detail you already gave to me, I gonna look at it for sure.

Please note that maintenance and contribution are made on my personal time off, so do not expect to high reactivity.

@KlfJoat
Copy link
Author

KlfJoat commented Feb 22, 2024

Please note that maintenance and contribution are made on my personal time off, so do not expect to high reactivity.

I understand. I appreciate your willingness to consider this enhancement.

Again, please let me know if I can assist in any way. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants