You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While I was looking into CI outputs, there was printed lines for the version of the shellcheck. However, while I was looking into .bash files I noticed some anti-patterns such as commands running with backticks etc. So, it is not utilised.
Also, When I run shellcheck against .bash files, I see quite few errors. I think it could be a good practice to run shellcheck both on CI tests and as a pre-commit hook.
Creating this issue for starting a discussion but I can definitely take the task for myself and go ahead.
The text was updated successfully, but these errors were encountered:
While I was looking into CI outputs, there was printed lines for the version of the
shellcheck
. However, while I was looking into.bash
files I noticed some anti-patterns such as commands running with backticks etc. So, it is not utilised.Also, When I run shellcheck against
.bash
files, I see quite few errors. I think it could be a good practice to runshellcheck
both on CI tests and as apre-commit
hook.Creating this issue for starting a discussion but I can definitely take the task for myself and go ahead.
The text was updated successfully, but these errors were encountered: