search

jumanjihouse / pre-commit-hooks

git pre-commit hooks that work with http://pre-commit.com/
MIT License
114 stars 52 forks source link

[feature] shellcheck and shfmt should avoid zsh #91

Open Freed-Wu opened 2 years ago

Freed-Wu commented 2 years ago

Shellcheck and shfmt don't support zsh. How about avoid passing zsh file to shellcheck and shfmt hooks?

Thanks!

WardsParadox commented 2 years ago

Seems like the hook does not include any ability to exclude and ignores config to exclude as well. Thinking this is an issue with #40 as well?

Freed-Wu commented 10 months ago

Seems like the hook does not include any ability to exclude and ignores config to exclude as well

I have tested #109 will ignore zsh. Can anyone review it?