search

ludeeus / action-shellcheck

GitHub action for ShellCheck.
MIT License
292 stars 69 forks source link

Updated matchers to account for "./" file prefixes #42

Closed fearphage closed 2 years ago

fearphage commented 3 years ago

The files are shown with a ./ prefix so they fail to string match the files on the disk. This updates the regex to account for this prefix.

Sample output:

##[debug]Checking ./catpants.sh
##[debug]Checking ./deploy-all-the-things.sh
Error: ./deploy-all-the-things.sh:116:6: error: Couldn't parse this test expression. Fix to allow more checks. [SC1073]
Error: ./deploy-all-the-things.sh:127:30: error: Expected test to end here (don't wrap commands in []/[[]]). Fix any mentioned problems and try again. [SC1072]
##[debug]Checking ./foo-bar

Since ./deploy-all-the-things.sh does not match deploy-all-the-things.sh, it would not annotate the file.

fearphage commented 3 years ago

@ludeeus Bump.

fearphage commented 3 years ago

@ludeeus Another attempt at getting your attention.