Darko Poljak
3be1c4ad6f
If nothing found shellcheck targets should succeed.
6 years ago
Darko Poljak
edaf4b46a5
++changelog
6 years ago
Darko Poljak
cdfa2dc628
Merge pull request #710 from darko-poljak/shellcheck-makefile-target
...
Shellcheck makefile target
6 years ago
Darko Poljak
140518f1b0
Add comment for SHELLCHECK_SKIP.
6 years ago
Darko Poljak
8436f872a9
Add shellcheck makefile target. shellcheck when doing release.
6 years ago
Darko Poljak
877b4d2748
++changelog
6 years ago
Darko Poljak
a46016242b
Merge pull request #708 from darko-poljak/shellcheck-various
...
Fix for various problems (shellcheck)
6 years ago
Darko Poljak
f039b0e6e5
Fix SC2030,SC2031.
6 years ago
Darko Poljak
68fad03a6b
Fix SC2120, SC2119.
6 years ago
Darko Poljak
b284bafbee
grep directly on the explorer file
6 years ago
Darko Poljak
02f39bcf6f
Fix SC2028.
6 years ago
Darko Poljak
0d80b4473b
Disable SC2031 for the single incident.
6 years ago
Darko Poljak
9a61aab631
++
6 years ago
Darko Poljak
88d4c8a059
Merge pull request #709 from darko-poljak/SC2044
...
Fix SC2044.
6 years ago
Darko Poljak
bf270b6f9c
Fix SC2044.
6 years ago
Darko Poljak
a369acea72
Fix SC2021.
6 years ago
Darko Poljak
644ec85d80
Fix SC2162.
6 years ago
Darko Poljak
16159d8fca
Fix SC2145.
6 years ago
Darko Poljak
596f3cf227
Disable SC2119,SC2120 where appropriate.
6 years ago
Darko Poljak
9b567ed8f7
Fix SC2230.
6 years ago
Darko Poljak
8cea8e67aa
Disable SC2031 where appropriate.
6 years ago
Darko Poljak
b120a813a2
Disable SC2030 where appropriate.
6 years ago
Darko Poljak
7b17ad4733
Fix SC2143.
6 years ago
Darko Poljak
e46420e7f5
Fix SC1090.
6 years ago
Darko Poljak
2f7b8e6db9
Fix SC2153.
6 years ago
Darko Poljak
73d0a5bbbe
Fix SC2235.
6 years ago
Darko Poljak
73acb35a8a
Fix SC2196.
6 years ago
Darko Poljak
28889ef38a
Disable SC2028 where intended.
6 years ago
Darko Poljak
ed542cc7cb
Fix SC2046.
6 years ago
Darko Poljak
b25aff17c6
Disable SC2039 where appropriate.
6 years ago
Darko Poljak
a601fa91d9
Fix SC2116.
6 years ago
Darko Poljak
5f8c631e34
++changelog
6 years ago
Darko Poljak
678ce2bfaa
Merge pull request #707 from darko-poljak/SC2016
...
Disable SC2016 check.
6 years ago
Darko Poljak
3b6d759660
Disable SC2016 check.
6 years ago
Darko Poljak
d2f1d6185d
++changelog
6 years ago
Darko Poljak
0cb127822f
Merge pull request #706 from darko-poljak/SC2164
...
Fix SC2164
6 years ago
Darko Poljak
49bb041330
Merge pull request #696 from thriqon/shellcheck-party-SC2086
...
Fix SC2086 on all scripts
6 years ago
Darko Poljak
c01cf38660
Fix SC2164
6 years ago
Darko Poljak
738f053877
++changelog
6 years ago
Darko Poljak
68b6477fe3
Merge pull request #705 from thriqon/shellcheck-party-various
...
Fix various shellcheck issues (SC2215, SC2059, SC2155, SC2091)
6 years ago
Darko Poljak
f0fb5215e8
Merge pull request #700 from thriqon/shellcheck-party-SC2034
...
Remove unused variables (SC2034)
6 years ago
Darko Poljak
3ead8b266a
Merge pull request #704 from thriqon/shellcheck-party-SC2069
...
Fix SC2069 (wrong stdout/stderr redirect order)
6 years ago
Jonas Weber
34b571b7d9
Fix SC2091 (execute result of condition)
...
For some reason, the echo|grep pattern was enclosed four times in a $()
operation. Nothing happened, since grep is always invoked with -q
(quiet, nothing printed to stdout).
6 years ago
Jonas Weber
bcd7fbbcc4
Fix SC2155 (directly export computed variables)
6 years ago
Jonas Weber
a0db375f91
Disable SC2059 (printf injection) for staged_file
...
We're explicitly allowing for 'printf injection' here, so it's ok to
just disable the corresponding shellcheck check.
6 years ago
Jonas Weber
a6fdf4255a
Fix SC2069 (wrong stdout/stderr redirect order)
...
In the original order, stderr was connected to the old stdout
(terminal). This was _probably_ not intended. The new order fixes this
by first connecting stdout to /dev/null and then attaching stderr to
that as well.
6 years ago
Jonas Weber
e2593c0dcb
Fix SC2215 (missing brackets on if)
...
This might have never worked as intended, but how would you know?
6 years ago
Jonas Weber
fff656d1ac
Unquote $recursive to support empty-case
...
This is safe to do, as the value of $recursive is not dependent on
user-supplied code.
6 years ago
Jonas Weber
53728a7ff0
Change backslash-dquote to single quote
...
Slightly breaking change, but improves security and readability.
6 years ago
Jonas Weber
ae7ccc5995
Fix SC2086 on all scripts
...
Relevant documentation: https://github.com/koalaman/shellcheck/wiki/SC2086
relates to: #540
This commit addresses 241 separate cases of missing quotes around
variables.
6 years ago