falseinstead of empty
if [ ] then echo "Temporarily disabled" fi
if false then echo "Temporarily disabled" fi
[ ] is a somewhat obscure way of expressing falsehood, and the behavior is likely intended to allow the incorrectly quoted command
[ $var ] to still work when the variable is unset.
POSIX has a more descriptive command
false for this.
None. This is a stylistic suggestion, and has no effect on how the script works.
ShellCheck is a static analysis tool for shell scripts. This page is part of its documentation.