Actionlint issue with Github actions

Question

I'm trying to use reviewdog/action-actionlint:v1.18.2 and I'm getting this error:

 shellcheck reported issue in this script: SC2046:warning:1:35: Quote this to prevent word splitting [shellcheck]

at this line:

run: |
          echo "COMMIT_COUNT=$(git rev-list $(git describe --tags --abbrev=0)..HEAD --count)" >> $GITHUB_ENV

what should I do ?

Paste the line into https://www.shellcheck.net/ to see where exactly it wants you to change things. — Benjamin W., Jan 04 '22 at 18:44
I followed the suggestions created by shellcheck.net and it worked! Thanks — Marcos Paulo Júnior, Jan 04 '22 at 20:26
The 1:35 means line 1 of the run section and column 35 is where the problem it is complaining about lies. The columns start even with the multiline "run" section. — Lee Meador, Jul 21 '23 at 23:21

score 1 · Answer 1 · answered Jan 05 '22 at 04:07

It's saying that it wants you to replace this:

echo "COMMIT_COUNT=$(git rev-list $(git describe --tags --abbrev=0)..HEAD --count)" >> $GITHUB_ENV

With this:

echo "COMMIT_COUNT=$(git rev-list "$(git describe --tags --abbrev=0)"..HEAD --count)" >> $GITHUB_ENV

This is so that if the output of git describe --tags --abbrev=0 contains a space for some reason, a space will end up in the argument to git rev-list, instead of two arguments being passed. I don't think the output of that particular command will ever actually contain a space, but it's a good practice to quote anyway unless you actually want splitting to happen.

Actionlint issue with Github actions

1 Answers1