anordal / shellharden

The corrective bash syntax highlighter
Mozilla Public License 2.0
4.62k stars 129 forks source link

Write script-beginning-line with double quotes #10

Closed j3k0 closed 6 years ago

j3k0 commented 6 years ago

So shellcheck doesn't complain about it.

$shellcheck demo.sh 
In demo.sh line 2:
if test "$BASH" = "" || "$BASH" -uc 'a=();true "${a[@]}"' 2>/dev/null; then
                                    ^-- SC2016: Expressions don't expand in single quotes, use double quotes for that.
anordal commented 6 years ago

Thanks for the tip!

Sorry, I didn't see that this was actually a pull request, but I fixed it exactly like you did.