Shellcheck fixes#99

Open

xavierba wants to merge 17 commits into

ltb-project:masterfrom

xavierba:shellcheck_fixes

Commits on May 11, 2026

SC2236 (style): Use -n instead of ! -z
xavierba
committed
SC2006 (style): Use $(...) notation instead of legacy backticks `...`
xavierba
committed
SC2086 (info): Double quote to prevent globbing and word splitting
xavierba
committed
SC2166 (warning): Prefer [ p ] || [ q ] as [ p -o q ] is not well defined
xavierba
committed
SC2059 (info): Don't use variables in the printf format string. Use printf '..%s..'
xavierba
committed
SC2126 (style): Consider using 'grep -c' instead of 'grep|wc -l'
xavierba
committed
SC2046 (warning): Quote this to prevent word splitting
xavierba
committed
SC2070 (error): -n doesn't work with unquoted arguments. Quote or use [[ ]]
xavierba
committed
SC2219 (style): Instead of 'let expr', prefer (( expr ))
xavierba
committed
SC2016 (info): Expressions don't expand in single quotes, use double quotes for that
xavierba
committed
SC2004 (style): $/${} is unnecessary on arithmetic variables
xavierba
committed
SC1090 (warning): ShellCheck can't follow non-constant source. Use a directive to specify location
xavierba
committed
SC2034 (warning): OS appears unused. Verify use (or export if used externally)
xavierba
committed
SC2115 (warning): Use "${var:?}" to ensure this never expands to /*
xavierba
committed
SC2003 (style): expr is antiquated. Consider rewriting this using $((..)), ${} or [[ ]]
xavierba
committed
SC2268 (style): Avoid x-prefix in comparisons as it no longer serves a purpose
xavierba
committed
SC2181 (style): Check exit code directly with e.g. 'if mycmd;', not indirectly with 0
xavierba
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Shellcheck fixes#99

Shellcheck fixes#99
xavierba wants to merge 17 commits into
ltb-project:masterfrom
xavierba:shellcheck_fixes

Commits on May 11, 2026

SC2236 (style): Use -n instead of ! -z

SC2006 (style): Use $(...) notation instead of legacy backticks `...`

SC2086 (info): Double quote to prevent globbing and word splitting

SC2166 (warning): Prefer [ p ] || [ q ] as [ p -o q ] is not well defined

SC2059 (info): Don't use variables in the printf format string. Use printf '..%s..'

SC2126 (style): Consider using 'grep -c' instead of 'grep|wc -l'

SC2046 (warning): Quote this to prevent word splitting

SC2070 (error): -n doesn't work with unquoted arguments. Quote or use [[ ]]

SC2219 (style): Instead of 'let expr', prefer (( expr ))

SC2016 (info): Expressions don't expand in single quotes, use double quotes for that

SC2004 (style): $/${} is unnecessary on arithmetic variables

SC1090 (warning): ShellCheck can't follow non-constant source. Use a directive to specify location

SC2034 (warning): OS appears unused. Verify use (or export if used externally)

SC2115 (warning): Use "${var:?}" to ensure this never expands to /*

SC2003 (style): expr is antiquated. Consider rewriting this using $((..)), ${} or [[ ]]

SC2268 (style): Avoid x-prefix in comparisons as it no longer serves a purpose

SC2181 (style): Check exit code directly with e.g. 'if mycmd;', not indirectly with 0